Hi Stuart,
Looks fine.
Roger
On 11/20/2014 6:51 PM, Stuart Marks wrote:
Hi all,
Here's a small test cleanup fix for review. This test fails rarely but
often enough to be annoying. The cause is one or more of the
following: a memory visibility issue (unsafe read/write to
non-volatile fields), a race condition (fixed Thread.sleep(500) wait
time), and potential failure to cancel a timer thread if an exception
is thrown. This change fixes all of these probems.
Webrev:
http://cr.openjdk.java.net/~smarks/reviews/8056313/webrev.0/
Bug:
https://bugs.openjdk.java.net/browse/JDK-8056313
Thanks,
s'marks
