On Mon, 16 Jul 2001, Doug MacEachern wrote:

> On Mon, 16 Jul 2001, Stas Bekman wrote:

> > - do we need modperl_use_module() to implement CORE::use
>
> into what namespace would things be imported?

true, no import() is needed other than for modules that use import() to do
various initialization/configuration staff, but these can be called from
startup.pl/<Perl> sections. no prob.

> > - should it be based on modperl_require_module() and convert the bareword
> >   into a filename before calling  modperl_require_module() or implemented
> >   from scratch
> >   s|::|/|;
> >   s|$|.pm|;
>
> they're both so small (see 1.xx's version), i wouldn't worry about it much
> for now.  they can be optimized later, like using Perl_load_module() for
> modperl_require_module().

ok

> > Sure, will fix. I thought I could actually take a benefit of using one
> > statement/no_braces_required C feature here :)
>
> i've learned to like how Perl forces the braces, easier to read and less
> error-prone :)

got it :)

> > > > +    MP_TRACE_d(MP_FUNC, "%s\n", arg);
> > >
> > > > +    MP_TRACE_d(MP_FUNC, "%s\n", arg);
> > >
> > > should be more that "%s\n" here.
> >
> > But you actually have only a single arg. Or are you talking about flagging
> > whether we are in parent process or vh scope? I've copied it from
> > MP_CMD_SRV_DECLARE(options) which traces only the options passed.
>
> the options command tells you a bit more:
>     MP_TRACE_d(MP_FUNC, "arg = %s\n", arg);
>
> that's all i was looking for.

oops :)


_____________________________________________________________________
Stas Bekman              JAm_pH     --   Just Another mod_perl Hacker
http://stason.org/       mod_perl Guide  http://perl.apache.org/guide
mailto:[EMAIL PROTECTED]   http://apachetoday.com http://eXtropia.com/
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/



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

Reply via email to