<Swing Dev> RFR: 8147544: Remove sun.misc.ManagedLocalsThread from java.desktop

[Chris Hegarty]

On 1 Apr 2016, at 23:19, Phil Race <philip.race at oracle.com> wrote:

> Bug: https://bugs.openjdk.java.net/browse/JDK-8147544
> Fix: http://cr.openjdk.java.net/~prr/8147544/

Your changes look ok to me.  Thanks for doing this Phil.

-Chris.

> ManagedLocalThreads is now superseded by a standard
> constructor on java.lang.Thread :-
> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/f90cd2c011bc
> 
> The various places in JDK that use the internal API need to migrate to this.
> 
> This fix addresses all (I hope) such usages in java.desktop.
> 
> I had to invent names for threads in numerous places since
> the required constructor insists on one.
> 
> Most uses install a Runnable but in about 4 cases we need to extend Thread.
> In those cases I added a comment, and in some cases a private no-args
> constructor() to try to guard against anyone coming along later and accidentally
> adding & calling a wrong super-class constructor.
> 
> JPRT has been used to verify the build on all platforms and I am
> in the process of running some sanity checks on the major
> platforms to make sure nothing unexpected happens but
> in general real testing of all of these changes is going to only happen
> with a nearly full test run, so the earlier the better to get the changes in.
> 
> -phil.
> 
>