<AWT Dev> [7u] Review Request for JDK-8006634 Unify LWCToolkit.invokeAndWait() and sun.awt.datatransfer.ToolkitThreadBlockedHandler
anthony.petrov at oracle.com
Mon May 6 05:52:04 PDT 2013
I agree. Please file a new CR for this work.
So, the current fix looks good to me then.
On 05/06/13 16:49, Sergey Bylokhov wrote:
> Hi, Anthony.
> I suggest to fix it via separate CR because of:
> - Shared code is not always ready to the situation, when the exception
> is thrown with locked
> - The same approach is used on windows too.
> - It would be good to have straightforward backports, when possible.
> On 30.04.2013 17:02, Anthony Petrov wrote:
>> Hi Petr,
>>> 45 unlock();
>>> 46 LWCToolkit.doAWTRunLoop(awtRunLoopMediator, processEvents);
>>> 47 lock();
>> I suggest to use the try/finally pattern for the unlock()/lock() calls
>> here to ensure the method always returns with the lock acquired.
>> Perhaps the same changes need to be applied to JDK 8, too.
>> The fix looks good otherwise.
>> best regards,
>> On 04/29/2013 05:56 PM, Petr Pchelko wrote:
>>> Hello, AWT Team.
>>> Please, review the back-port of the fix for JDK-8006634 to the JDK7u.
>>> I am sending it as a separate review request because the code changes
>>> are different in the patches.
>>> The bug:
>>> The webrev for JDK7-u:
>>> The original webrev for JDK-8
>>> The mailing thread:
>>> The idea of the fix stays the same. However the code changes are
>>> different, because it is affected by the fix for 7171163 made by
>>> Alexander Zuev. His fix was different for JDK8 and JDK7. (see
>>> LWCToolkit.m, LWCToolkit.java) However, after those fix the
>>> differences will be eliminated.
>>> With best regards. Petr.
More information about the jdk7u-dev