[jira] [Commented] (PIVOT-969) Better handle uncaught exceptions in background threads in Pivot

["Sandro Martini (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PIVOT-969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14550199#comment-14550199 ] 

Sandro Martini commented on PIVOT-969:
--------------------------------------

The patch to me looks good ... go with the commit.
Would it be possible to merge even in 2.0.x ?

Later, if you find another (better) solution even that breaks compatibility (only in 2.1), be free to revise/update ...
Thanks.


> Better handle uncaught exceptions in background threads in Pivot
> ----------------------------------------------------------------
>
>                 Key: PIVOT-969
>                 URL: https://issues.apache.org/jira/browse/PIVOT-969
>             Project: Pivot
>          Issue Type: Improvement
>          Components: wtk
>    Affects Versions: 2.0.4, 2.1
>         Environment: All
>            Reporter: Roger Whitcomb
>            Assignee: Roger Whitcomb
>            Priority: Minor
>              Labels: exception-reporting, uncaught-exceptions
>             Fix For: 2.1
>
>         Attachments: task.patch
>
>
> PIVOT-916 regularized the handling of uncaught exceptions in most places in the GUI thread and during startup.  But background threads can still basically ignore uncaught exceptions.  But using the Thread.setDefaultUncaughtExceptionHandler and changing some of our interfaces (namely Application.UncaughtExceptionHandler) we could use the ApplicationContext.handleUncaughtException as the default exception handler (which delegates to an application's override method).
> This would involve changing the signature (or adding an override method) that accepts (Thread, Throwable) in both ApplicationContext and Application.UncaughtExceptionHandler interface.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)