[BATIK-739] Suggestion for clarification in UpdateManager.interrupt comment

• Type: Bug
• Status: Open
• Resolution: Unresolved
• Affects Version/s: 1.7
• Component/s: Bridge
• Environment: Operating System: Linux; Platform: PC
• Assignee: Batik Developer's Mailing list
• Reporter: Jeff Grigg
• Created: 11/May/07 13:48
• Updated: 08/Nov/12 11:10

Description

A suggestion for additional information that could be included to explain the
empty catch clause in the UpdateManager.interrupt comment:

// An IllegalStateException can be thrown by the RunnableQueue.resumeExecution()
method, due to a race condition: The RunnableQueue.preemptLater method call
starts the process of killing the RunnableQueue thread, which could finish
before the resumeExecution method is called. Additional synchronization might
fix this, but it's easy and effective to discard the redundant exception.

(I ran into this while tracking real IllegalStateException cases caused by our
code. It took a little research to determine that this IllegalStateException is
NOT a problem. So it might save other developers some confusion and effort if
additional comments were present.)