Re: Please help adding ModPerl::Interpreter

2007-10-22 Thread Torsten Foertsch
On Monday 22 October 2007, Philippe M. Chiasson wrote: > And make modperl_trace always prefix log messages with modperl_tid_pid()? That makes sense. I didn't know about modperl_pid_tid(). But the implementation is not good. 1) It requires a pool that is not always available 2) It introduces a me

Re: Please help adding ModPerl::Interpreter

2007-10-22 Thread Philippe M. Chiasson
Torsten Foertsch wrote: > On Thursday 18 October 2007, Philippe M. Chiasson wrote: >> Any chance you can break the patch into multiple patches > > This one adds the current pid to the output of modperl_trace. Wouldn't a better all-around approach use a pid/tid combination for threaded MPMs and fa