This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Patch: initialize ProcessManager early

From: Tom Tromey <tromey at redhat dot com>
To: David Daney <ddaney at avtrex dot com>
Cc: GCJ-patches <java-patches at gcc dot gnu dot org>, Mark Wielaard <mark at klomp dot org>
Date: 16 Feb 2007 10:15:52 -0700
Subject: Re: Patch: initialize ProcessManager early
References: <m37iuj174i.fsf@localhost.localdomain> <45D5ED67.4090802@avtrex.com> <45D5EE76.1080408@avtrex.com>
Reply-to: tromey at redhat dot com

>>>>> "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.

Tom

Follow-Ups:
- Re: Patch: initialize ProcessManager early
  - From: David Daney

References:
- Patch: initialize ProcessManager early
  - From: Tom Tromey
- Re: Patch: initialize ProcessManager early
  - From: David Daney
- Re: Patch: initialize ProcessManager early
  - From: David Daney

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]