Modify

Opened 16 years ago

Closed 16 years ago

#2309 closed defect (fixed)

[PATCH] Exception might get lost when ExecutorService is used

Reported by: jttt Owned by: team
Priority: major Milestone:
Component: Core Version:
Keywords: Cc:

Description

When task is executed using ExecutorService, any exception is ignored unless Future.get is called. This may hide some problems in code.

Provided patch modifies PleaseWaitRunnable so that it throws RuntimeException in EDT when exception during task execution is encountered. This behaviour is optional in case somebody wants to handle exception by himself. The default is throw exception in EDT.

It is not the best solution because not every task has to inherit from PleaseAndWaitRunnable and also exception is shown as JOSM error even if it's actually plugin error.

Attachments (1)

executor_exception.patch (4.8 KB ) - added by jttt 16 years ago.

Download all attachments as: .zip

Change History (2)

by jttt, 16 years ago

Attachment: executor_exception.patch added

comment:1 by stoecker, 16 years ago

Resolution: fixed
Status: newclosed

In r1493.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.