New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stacktrace-printing hardcoded in PApplet.java #222

Closed
processing-bugs opened this Issue Feb 10, 2013 · 3 comments

Comments

Projects
None yet
3 participants
@processing-bugs

processing-bugs commented Feb 10, 2013

Original author: b...@processing.org (June 07, 2010 01:15:27)

This bug automatically added from:
http://dev.processing.org/bugs/show_bug.cgi?id=1281

Comment from taifunbrowser, 2009-06-16 20:44

To increase object-orientedness, could all

e.printStackTrace

be replaced with

printException(Throwable e)

in Papplet.java ? Then those of us wanting to extend the error handling capabilities
(i.e., if we have many registerDraws() that could throw exceptions), we could just
override this single method.

Original issue: http://code.google.com/p/processing/issues/detail?id=183

@processing-bugs

This comment has been minimized.

Show comment
Hide comment
@processing-bugs

processing-bugs Feb 10, 2013

From b...@processing.org on June 07, 2010 01:15:28
Comment from fry, 2009-06-17 03:12

yeah, this is on the todo list but didn't have a bug number assigned to it.
we do this with the die() method that handles fatal exceptions, but i need
to make the implementation more consistent, and add a warn() method that
would handle cases like the one you describe.

processing-bugs commented Feb 10, 2013

From b...@processing.org on June 07, 2010 01:15:28
Comment from fry, 2009-06-17 03:12

yeah, this is on the todo list but didn't have a bug number assigned to it.
we do this with the die() method that handles fatal exceptions, but i need
to make the implementation more consistent, and add a warn() method that
would handle cases like the one you describe.

@benfry benfry removed the bug label Nov 15, 2014

@JakubValtar

This comment has been minimized.

Show comment
Hide comment
@JakubValtar

JakubValtar Aug 6, 2015

Contributor

There are still 43 matches for e.printStackTrace() in PApplet.java, can we label this as help?

Contributor

JakubValtar commented Aug 6, 2015

There are still 43 matches for e.printStackTrace() in PApplet.java, can we label this as help?

@JakubValtar JakubValtar added revised and removed revision needed labels Aug 6, 2015

@benfry benfry self-assigned this Aug 11, 2015

@JakubValtar JakubValtar removed the revised label Aug 11, 2015

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Aug 13, 2016

Member

Added printStackTrace(Throwable t) for 3.2.

Member

benfry commented Aug 13, 2016

Added printStackTrace(Throwable t) for 3.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment