Re: AIM/mp2 Apache::compat bug?

[prev] [thread] [next] [Date index for 2004/11/23]

Malcolm J Harwood wrote:
> On Tuesday 23 November 2004 11:56 am, Stas Bekman wrote:
> 
> 
>>>>with mp2 you need to update the stat info yourself each time you update
>>>>$r->filename. something like this:
>>>>
>>>> $r->filename($newfile);
>>>> $r->finfo(APR::Finfo::stat($newfile, APR::FINFO_NORM, $r->pool));
> 
> 
>>I've updated the docs to mention that:
>>http://perl.apache.org/docs/2.0/api/Apache/RequestRec.html#C_filename_
> 
> 
> Out of curiosity, what was the reason for the change? I would have thought it 
> is much more prone to user error (people not knowing or forgetting to update 
> the states). Are there cases where you'd change the filename but not need to 
> restat it?

That's really a question to httpd developers, mod_perl just provides the 
perl API for the Apache C API.

Granted we could do that behind the scenes, but it's not always the 
desired behavior so I'm not sure we should. May be an option argument 
could be passed to do the work?


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

-- 
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html