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

Tom Tromey wrote:
"Andrew" == Andrew Haley <aph@redhat.com> writes: 

Andrew> You have to wonder whether Java's Process / Streams interface is
Andrew> really a sensible way to spawn a process that does name lookup for
Andrew> stack traces .  An earlier version of this name lookup code was
Andrew> written in C++ and didn't use Process at all.

I don't remember why this change was made.  Windows support maybe?
Also I thought we had to do all process reaping in a single thread?

Some pthread implementations (linux-threads) require that process reaping occurs in the same thread that forked the process. That was my motivation for the original ProcessManager patch.

The reasons for the ProcessManager patch are of course unrelated to any motivations for changes in the way stack trace line numbers are generated.

David Daney

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