Re: Code Review Request for CR 7143070 - test/java/awt/print/PaintSetEnabledDeadlock/PaintSetEnabledDeadlock.java freezes on exit

[Anthony Petrov]

Hi Oleg,

Thanks for the webrev!

  60         if(!ret)
  61             throw new RuntimeException("Test failed!");

Please add a space after 'if' and put the throw statement into a {} block.

Otherwise the fix looks good to me.

--
best regards,
Anthony

On 2/8/2012 4:56 PM, Oleg Pekhovskiy wrote:

Hi guys,

here is the fix for:
http://bugs.sun.com/view_bug.do?bug_id=7143070

webrev:

http://cr.openjdk.java.net/~denis/7143070/webrev.03/ <http://cr.openjdk.java.net/%7Edenis/7143070/webrev.03/>

The main idea was eliminate test freezing on exit for some systems.
So synchronization between threads on exit was rewritten.

Thanks,
Oleg