www-apache-bugdb mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dean Gaudet <dgau...@arctic.org>
Subject Re: documentation/789: Bad Perl Code is mentioned in the FAQ, #31
Date Thu, 26 Jun 1997 10:55:11 GMT
/*nph- doesn't match /foonph-cgi ... it matches things like /////nph-.
Which seems pretty correct to me.  Whereas ^/*nph- won't match an nph
with a full path.

Dean

On Thu, 26 Jun 1997 coar@hyperreal.com wrote:

> Synopsis: Bad Perl Code is mentioned in the FAQ, #31
> 
> State-Changed-From-To: open-closed
> State-Changed-By: coar
> State-Changed-When: Thu Jun 26 02:23:08 PDT 1997
> State-Changed-Why:
> >a) the regex is wrong as it allows /foonph-cgi (due to the /*).
> 
> Right you are, though the correct RE is m:^/*nph-:.  Fixed.
> 
> >b) printf is not needed. print should be used.
> 
> Using printf is not an error, but you're right - print
> is definitely better here.  Fixed.
> 
> >c) no semi colon should be after the if statement.
> 
> This isn't an error, either, but essentially the same
> reasoning applies.  Fixed.
> 
> These changes will appear on the web site shortly, and
> will be in the documentation included in the next release.
> Thank you for pointing this out, and for using Apache!
> 
> 


Mime
View raw message