JDK RFR of 8147545: Remove sun.misc.ManagedLocalsThread from java.prefs

[ecki at zusammenkunft.net]

Hello,

 I would name it more like "Preference Timer Shutdown Hook" and "Preference Dispatch Thread" and "Preference Flush Thread" (i.e. include the subsystem)

+ new Thread(null, null, "Sync Timer Thread", 0, false)

Gruss
Bernd

-- 
http://bernd.eckenfels.net

-----Original Message-----
From: Chris Hegarty <chris.hegarty at oracle.com>
To: Brian Burkhalter <brian.burkhalter at oracle.com>
Cc: core-libs-dev <core-libs-dev at openjdk.java.net>
Sent: Fr., 22 Jan. 2016 21:38
Subject: Re: JDK RFR of 8147545: Remove sun.misc.ManagedLocalsThread from java.prefs

On 22 Jan 2016, at 20:17, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:

> Please review at your convenience.
> 
> Issue:	https://bugs.openjdk.java.net/browse/JDK-8147545
> Patch:	http://cr.openjdk.java.net/~bpb/8147545/webrev.00/

Looks good to me. Thanks Brian.

-Chris.

> Replaced using of ManagedLocalsThread with a Thread created from the new constructor
> 
> Thread(ThreadGroup,Runnable,String,long,boolean)
> 
> with the boolean parameter “inheritThreadLocals” set to false.
> 
> Thanks,
> 
> Brian