Re: [mp2 Patch] ModPerl::Util::undef(coderef)

On Thu, 2003-06-19 at 15:50, Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> > On Thu, 2003-06-19 at 14:21, Stas Bekman wrote:
> > 
> >>Philippe M. Chiasson wrote:
> >>
> >>>Following the discussion on Apache::Reload and Apache::Symbol, I've
> >>>taken out Apache::Symbol::undef from mp1, massaged it a bit and made it
> >>>into ModPerl::Util::undef for mod_perl 2.
> >>>
> >>>Works wonders when you want to undefine subroutines, constant or not,
> >>>with or without prototypes. Very usefull for Apache::Reload (can get rid
> >>>of that $SIG{__WARN__} trap).
> >>
> >>As suggested in the other thread, ModPerl::Util::undef is not the best 
> >>choice 
> >>for the name. Me thinking either call ModPerl::Util::symbol_undef or have 
> >>further subclassing: ModPerl::Util::Symbol::undef
> > 
> > 
> > How about like I said before ModPerl::Util::undef_code since it works
> > only on CODE refs?
> 
> but we might have others as well, so a more generic name is better, no? 
> symbol_undef_code? just in case?

Sure, after all, it's just a name ;-) I am fine with calling it
symbol_undef_code. 

s/undef/symbol_undef_code/ && commit ?

> >>Otherwise +1, but again please first ask Doug why hasn't he ported this 
> >>code 
> >>in first place.
> > 
> > 
> > Will do ;-)
> > 
> > 
> >>__________________________________________________________________
> >>Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
> >>http://stason.org/     mod_perl Guide ---> http://perl.apache.org
> >>mailto:stas@xxxxxxxxxx http://use.perl.org http://apacheweek.com
> >>http://modperlbook.org http://apache.org   http://ticketmaster.com
-- 
--------------------------------------------------------------------------------
Philippe M. Chiasson /gozer\@(cpan|ectoplasm)\.org/ 88C3A5A5 (122FF51B/C634E37B)
http://gozer.ectoplasm.org/    F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3 A5A5
Q: It is impossible to make anything foolproof because fools are so ingenious.
perl -e'$$=\${gozer};{$_=unpack(P7,pack(L,$$));/^JAm_pH\n$/&&print||$$++&&redo}'

signature.asc
Description: This is a digitally signed message part