[#AMQ-891] InterruptedException handling tweaks - Apache ActiveMQ & Apache ServiceMix

ActiveMQ

InterruptedException handling tweaks

Created: 22/Aug/06 12:50 AM Updated: 03/Sep/06 10:47 PM

Component/s:

Broker

Affects Version/s:

4.0.2

Fix Version/s:

4.1.0, 4.0.2

Time Tracking:

Not Specified

File Attachments:

	File Name	Attached Date	Attached By	Size
	interrupted.patch.gz	2006-08-22 12:50 AM	John Heitmann	1 kb

Patch Info:

Patch Available

Description

« Hide

There were a few spots where the broker was masking the interrupt state after handling an InterruptedException. This is a lint pass to clean some of that up. I learned after I made this patch that it's actually slightly better stylistically to call Thread.interrupt() instead of Thread.currentThread().interrupt() since it's static and the interrupt state is global(ish), but this is the version we've tested.

All

Comments

Work Log

Change History

Subversion Commits

FishEye

Crucible

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

James Strachan - 25/Aug/06 08:42 AM

Do you wanna handle this one - I seem to remember you doing some work on the interupted stuff?

[ Permlink | « Hide ]

Hiram Chirino - 03/Sep/06 10:47 PM

Great patch! Applied to trunk rev 439886 and to 4.0 branch rev 439941