On Wed, 27 Sep 2000, Salvador Ortiz Garcia wrote:
 
> The 'push_array(cls->Perl...' in both cases can be put inside an else part
> for the 'if(PERL_RUNNING()) {' to make clear that the process were
> delayed.

good idea.
 
> btw, when i was making the patch, i realize that right now the inline
> PerlRequire case (not PerlModule) is broken respect to processing
> %ApacheReadConfig:: for PERL_SECTIONS, take a close look, right now it
> reads:
... 
> isnt executed. (And executed in the delayed case, when not needed)
> 
> Full path attached.        

applied, thanks!
i made a few style changes to it though, i've been trying to do better
about following:
http://dev.apache.org/styleguide.html

in particular, no tabs for indenting.  there's lots of tabs in the
sources, in part because i didn't figure out until somewhat recently the
.emacs magic to get rid of them: (setq indent-tabs-mode nil)
i don't know what the vi magic is.
 
> Yes, PerlEval is much better, and with it they can say:
> 
> PerlEval do foo 

coolness.
 
> Returning to mod_perl hacking.

great news, welcome back!!


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to