Le 21/11/2011 14:05, Alan Bateman a écrit :
On 21/11/2011 12:45, Alan Bateman wrote:
On 21/11/2011 12:17, Gary Adams wrote:
Here's a fresh bug number to handle the volatile revision (pun intended) 7114125 java/util/Timer/KillThread.java should use volatile cross thread variable declaration

Here's a fresh webrev for the proposed adjusted fix.
  http://cr.openjdk.java.net/~gadams/7114125/

This looks good to me.
I see David has reviewed this too so I've pushed this change (minus the addition to the @bug tag). I've also pushed your fix to the ThreadGroup/Stop.java test so I think this means we're now caught up on all your contributions to these tests.

-Alan.

Alan, David and Gary,
I'm sorry to have proposed to use an array of Thread forgetting that
the array was accessed by more than one thread.

Rémi

Reply via email to