[7u8] Review request for 7189350: Fix failed for CR 7162144 (original) (raw)
Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon Aug 20 09:01:57 PDT 2012
- Previous message: [7u8] Review request for 7189350: Fix failed for CR 7162144
- Next message: [7u8] Review request for 7189350: Fix failed for CR 7162144
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Is it ok that we reset the interrupted status in the 251-253?
20.08.2012 19:24, Anthony Petrov wrote:
Hello,
Please review a fix for http://bugs.sun.com/bugdatabase/viewbug.do?bugid=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
-- Best regards, Sergey.
- Previous message: [7u8] Review request for 7189350: Fix failed for CR 7162144
- Next message: [7u8] Review request for 7189350: Fix failed for CR 7162144
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]