Patch: initialize ProcessManager early

David Daney ddaney@avtrex.com
Fri Feb 16 21:26:00 GMT 2007


Tom Tromey wrote:
>>>>>> "David" == David Daney <ddaney@avtrex.com> writes:
>>>>>>             
>
> David> New patch attached (still untested).
>
> If it works, it is fine by me.
>
> David> -	InheritableThreadLocal.newChildThread(this);
> David> +        if (!noInheritableThreadLocal)
> David> +          InheritableThreadLocal.newChildThread(this);
>
> A comment here explaining why would be useful.
> You could lift the comment from my patch.
>
>   
This is what I committed after regression testing on x86_64-pc-linux-gnu 
(FC6):

2007-02-16  David Daney  <ddaney@avtrex.com>

    * java/lang/Thread.java (Thread(ThreadGroup, Runnable, String)): Pass
    new parameter constructor.
    (Thread(ThreadGroup, Runnable, String, long)): Same.
    (Thread(String, boolean)): New constructor.
    (Thread(Thread, ThreadGroup, Runnable, String): Add parameter
    noInheritableThreadLocal, don't call
    InheritableThreadLocal.newChildThread if set.
    * java/lang/PosixProcess.java(ProcessManager()): Set
    noInheritableThreadLocal in super.
    * java/lang/natThread.cc (_Jv_AttachCurrentThread): Pass new
    parameter to Thread constructor.
    (_Jv_AttachCurrentThreadAsDaemon): Same.
    * java/lang/Thread.h: Regenerate.
    * classpath/lib/java/lang/Thread.class: Same.
    * classpath/lib/java/lang/PosixProcess$EOFInputStream.class: Same.
    * classpath/lib/java/lang/PosixProcess.class: Same.
    * classpath/lib/java/lang/Thread$State.class: Same.
    * classpath/lib/java/lang/PosixProcess$ProcessManager.class: Same.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: process-thread.diff
Type: text/x-patch
Size: 5712 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20070216/e21a6867/attachment.bin>


More information about the Java-patches mailing list