<AWT Dev> [7u8] Review request for 7189350: Fix failed for CR 7162144

Artem Ananiev artem.ananiev at oracle.com
Tue Aug 21 03:54:10 PDT 2012


Hi, Anthony,

since we discussed this fix offline last week, it obviously looks fine 
for me.

Thanks,

Artem

On 8/20/2012 7:24 PM, 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



More information about the awt-dev mailing list