RFR: 6402201: ProcessAttachTest.sh needs better synchronization

[Erik Gahlin]

Thanks, didn't know about ATOMIC_MOVE. it will be useful for other tests 
as well.

http://cr.openjdk.java.net/~egahlin/6402201_3/

Erik

Alan Bateman skrev 2013-11-20 12:43:
> On 20/11/2013 11:25, Erik Gahlin wrote:
>> Thanks for your feedback
>>
>> Here is an updated webrev where the file is renamed.
>> http://cr.openjdk.java.net/~egahlin/6402201_2/
> That looks okay to me. One small improvement would be to use 
> try-with-resources around the file write. Alternatively you could use 
> Files.write and do it in one method call (which would get you using 
> the new file system API and then you could use Files.move and specific 
> the ATOMIC_MOVE option). It's not strictly required for this and what 
> you have seems to fix the reported issue.
>
> -Alan