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.
