Looks fine to me.
Thanks,
Oleg
20.08.2012 19:24, Anthony Petrov wrote:
Hello,
Please review a fix for
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7189350 at:
http://cr.openjdk.java.net/~anthony/7u8-1-missingAWTThread-7189350.0/
With this fix we resolve an issue originally described in 7162144 by
properly handling thread interruption requests. We no longer override
the Thread.interrupt() for the EDT, and hence don't clash with
NetBeans using thread interruption for its own purposes.
Note that there's still not a regression test since NetBeans' test is
very heavy (over 100MB) and impractical for our purposes. However, the
NetBeans team has tested a developer build with this fix and found no
regressions.
PS. We aren't yet sure if we want the same fix for JDK 8, hence we fix
it for 7u8 first.
--
best regards,
Anthony
