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. >>> >>
