Exception handling in PleaseWaitRunnable

[bastiK]

In the PleaseWaitRunnable code the parameter ignoreException has the description If true, exception will be propagated to calling code.

However this is not true, the Exceptions are really ignored, i.e. simply dropped. The following code demonstrates the problem: It does not lead to any visible Exception.

    Main.worker.submit(new Runnable() {
        public final void run() {
            throw new RuntimeException();
       }
    });

In this thread they have a view ways to catch exceptions in such a case:
​http://stackoverflow.com/questions/1687977/how-to-properly-catch-runtimeexceptions-from-executors

Currently we catch all exceptions inside the Runnable. There are two kinds of Exceptions: (1) Errors that are expected (Parse error, io error) and (2) Coding errors. (1) should lead to a simple message box and (2) should open the bug report window.

Currently this is not the case: All errors except RuntimeException get a simple message box.

[anonymous]

The comment also says - When this runnable is executed using executor framework then use false unless you read result of task (because exception will get lost if you don't).

If ignoreException is set to false, exception will be caught and rethrown in EDT.

[bastiK]

Replying to anonymous:

The comment also says - When this runnable is executed using executor framework then use false unless you read result of task (because exception will get lost if you don't).

First part I agree.

If ignoreException is set to false, exception will be caught and rethrown in EDT.

Maybe this used to be the case, but currently it just shows a message box:

line 127:
            // Exception has to thrown in EDT to be shown to user
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    ExceptionDialogUtil.explainException(ex);
                }
            });

[jttt]

(In [2984]) Fix #4524 Exception handling in PleaseWaitRunnable

[jttt]

I've changed it in ExceptionDialogUtil to show bug report dialog when unknown exception is thrown.