perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philippe M. Chiasson" <>
Subject Re: Please help adding ModPerl::Interpreter
Date Thu, 25 Oct 2007 07:43:25 GMT
Torsten Foertsch wrote:
> On Thursday 18 October 2007, Philippe M. Chiasson wrote:
>> Any chance you can break the patch into multiple patches
> This one registers the cleanup phase with a subpool to ensure it is run before 
> pnotes are destroyed. Subpools are destroyed first thing in 
> apr_pool_{clear,destroy}. Hence, a pool cleanup registered with a subpool is 
> always called before a cleanup registered with the main pool.

Nice and elegant, and removes a piece of annoying, special-case code.

Applied to trunk/ as r588156.

Little style nit (see
for the whole details.

We globally use 4 spece indents, _not_ the \t (tab) character.

Philippe M. Chiasson     GPG: F9BFE0C2480E7680 1AE53631CB32A107 88C3A5A5       m/gozer\@(apache|cpan|ectoplasm)\.org/

View raw message