The fix looks good to me.
--
best regards,
Anthony
On 07/18/2013 03:20 PM, Petr Pchelko wrote:
Hello, AWT team.
It's a reminder about a very old review I have posted.
Could you please review a fix for the following issue:
http://bugs.sun.com/view_bug.do?bug_id=2227234
The fix is available at:
http://cr.openjdk.java.net/~pchelko/2227234/webrev.00/
This is a partial forward port of the fix made by Alexander Zyev in jdk7.
The original bug in 7 is:
http://bugs.sun.com/view_bug.do?bug_id=7184951
The original fix could be found at:
http://cr.openjdk.java.net/~kizune/7184951/webrev.00/
The changes in CToolkitThreadBlockedHandler are not needed after the fix for
JDK-8006634 Unify LWCToolkit.invokeAndWait() and CToolkitThreadBlockedHandler
Artem has already approved the fix, could I get a second review please?
With best regards. Petr.
On May 16, 2013, at 1:11 PM, Artem Ananiev wrote:
Looks fine.
Thanks,
Artem
On 5/16/2013 12:50 PM, Petr Pchelko wrote:
Hello, AWT Team.
This is a reminder. Could you please review this fix.
With best regards. Petr.
On Mar 14, 2013, at 2:54 PM, Petr Pchelko wrote:
Hello, AWT Team.
Could you please review a fix for the following issue:
http://bugs.sun.com/view_bug.do?bug_id=2227234
The fix is available at:
http://cr.openjdk.java.net/~pchelko/2227234/webrev.00/
This is a partial forward port of the fix made by Alexander Zyev in jdk7.
The original bug in 7 is:
http://bugs.sun.com/view_bug.do?bug_id=7184951
The original fix could be found at:
http://cr.openjdk.java.net/~kizune/7184951/webrev.00/
The changes in CToolkitThreadBlockedHandler are not needed after the fix for
JDK-8006634 Unify LWCToolkit.invokeAndWait() and CToolkitThreadBlockedHandler
With best regards. Petr.
