On 10/11/2016 4:36 PM, Alexander Zvegintsev wrote:


On 07.10.2016 17:07, Sergey Bylokhov wrote:
On 07.10.16 10:28, Semyon Sadetsky wrote:
Hi Sergey,

1525         queueEmpty = false;
1526         eventDispatched = false;
1527         synchronized(waitLock) {

Shouldn't lines 1525 & 1526 be moved inside the synchronized block?

I think so, at least this will unify how we use these in dispatch event and at the end of waitForIdle():

On 9/26/2016 8:38 PM, Sergey Bylokhov wrote:

Please review the fix for jdk9.

The SunToolkit.waitForIdle() uses string literal for synchronization,
which means some code (intentionally or not) can hang Robot.waitForIdle.

Bug: https://bugs.openjdk.java.net/browse/JDK-8166673
Webrev can be found at:

Reply via email to