Code review - 6731620 TEST_BUG: java/util/Timer/Args.java is too optimistic about the execution time of System.out.printf

[Gary Adams]

  On 11/21/11 05:44 AM, Gary Adams wrote:
> On 11/21/11 5:37 AM, Alan Bateman wrote:
>> On 21/11/2011 01:39, David Holmes wrote:
>>> Never mind - Alan already tracked down reviews and did push.
>> Yes, I pushed the three that I thought had been reviewed and thoroughly 
>> battered to death. There is a fourth one 
>> (test/java/lang/ThreadGroup/Stop.java) that I need to double check who the 
>> reviewers are before pushing it.
>>
>> On test/java/util/Timer/KillThread.java then I agree this should be changed 
>> as array elements cannot be volatile. I don't think it it will be an issue in 
>> practice but we might as well get it done now while the discussion is fresh.
>>
>> -Alan.
>>
>>
> I'll be glad to spin another change here.
> There were conflicting suggestions on
> this bug and I thought I took the last
> comment.
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/