Perrin Harkins wrote:
On Tue, 2004-01-20 at 15:09, Stas Bekman wrote:

Good.  Do you think there should be a separate module for this, that
just acts a shell for $r and AUTOLOADs other things?  That would avoid
loading Apache::RequestRec unless it is needed.  Apache::RequestHandle
maybe?

You don't necessarily need a module for that. All you want to do is to install


sub Apache::RequestRec::AUTOLOAD, with a few lines of code.


Why do it in a strange way?  Why not simply have a normal module?  It's
easier to maintain if things live in the most obvious place.

It's easier when there is one or two objects. I repeat that in mp2 there are *many* objects, and adding many new modules just to install an AUTOLOAD module which is going to be exactly the same with is not going to make things easier to maintain.


Really, it should have been UNIVERSAL::AUTOLOAD that will handle them all at once as you can see from the ModPerl::MethodLookup manpage. And it works for most phases, but the startup ones, where it mysteriously segfaults.

See, this is your "problem" - mp2 is much more than $r and $s. But you want to fix only those because you don't know about the others. We here try to have it right and homogenious for the whole API. So people who will want to use mp2 just to write filters won't say, why $r is easy but $f is not?


Good question.  Why wouldn't we do the same thing with $f, and any
others that I don't know about?

We would. And that's exactly why I didn't stop at the suggestion of yours for $r-problem.



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


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



Reply via email to