httpd-apreq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Schaefer <joe+gm...@sunstarsys.com>
Subject Re: cvs commit: httpd-apreq-2/src apreq_cookie.c
Date Thu, 01 Jul 2004 05:46:46 GMT
Stas Bekman <stas@stason.org> writes:

> joes@apache.org wrote:

[...]

> >   +    Perl_require_pv(aTHX_ "APR/Error.pm");
> 
> while copying you've lost the comment:
> 
> /* XXX: it'd be nice to arrange for it to load early */
> 
> I just haven't figured out yet how to make this happen early, due to
> Apache2/ subdir adjustment which kicks in when user wished it to
> happen.

I would imagine that this isn't a great concern for apreq, because
Apache::Request is already installed in the "right place".  We
should be able to just 'use APR::Error' in Request_pm etc and
drop that Perl_require_pv... line altogether.

> Otherwise it's a waste to try to load it all the time. Even though
> for an error case.

Agreed.
-- 
Joe Schaefer


Mime
View raw message