perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stas Bekman <s...@stason.org>
Subject Re: [Fwd: Re: [mp2][PATCH] enhancement for Apache::Reload]
Date Wed, 02 Jun 2004 20:31:48 GMT
Geoffrey Young wrote:
> 
> Stas Bekman wrote:
> 
>>Looks good Andreas, Thank you! I'm forwarding this to the dev list.
>>
>>Anybody has issues with this change? 
> 
> 
> assuming it works, it sounds fine.  I don't have the free tuits to test it atm.

Same here. I'll probably get to polish it once I've delivered my tutorial 
material.

>>I'd probably do a little tweak to
>>the code, but otherwise looks good. Might reuse Apache::TestTrace for
>>this functionality.
> 
> 
> well, you probably don't want to use Apache::TestTrace in code that exists
> outside of the test system since they are different projects - presumably we
> could remove the entire test framework and move to something else, but
> Apache::Reload would still require part of A-T to function, which would be
> kinda strange.

Right. But we could still copy most of the code from there. i.e. replace those 
explicit:

   print ... if $debug > ...;

with debug level wrappers functions, which I'm now used to and love them.

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Mime
View raw message