On 5/15/13 1:12 PM, Jaroslav Bachorik wrote:
Ok. I've changed it to use AtomicBoolean Thanks for pointing this out. Even though it's rather obvious I got so used to using the final array in anonymous inner classes that I didn't even stop and think. BTW, there are a lot of places in the tests using this idiom in potentially multithreaded environment :(
Hrrm. OK, well, let's fix one test at a time. :-)
Updated webrev - http://cr.openjdk.java.net/~jbachorik/8005472/webrev.10
Thanks for the update. I took a quick look at it seems fine now. I think Chris may need to be the OpenJDK Reviewer on this one. s'marks