Received: (from majordom@localhost)
	by hyperreal.com (8.8.5/8.8.5) id CAA13477;
	Thu, 26 Jun 1997 02:23:25 -0700 (PDT)
Received: (from nobody@localhost)
	by hyperreal.com (8.8.5/8.8.5) id CAA13466;
	Thu, 26 Jun 1997 02:23:09 -0700 (PDT)
Date: Thu, 26 Jun 1997 02:23:09 -0700 (PDT)
Message-Id: <199706260923.CAA13466@hyperreal.com>
To: apache-bugdb@apache.org, coar@apache.org, dion@member.com
From: coar@hyperreal.com
Subject: Re: documentation/789: Bad Perl Code is mentioned in the FAQ, #31
Sender: apache-bugdb-owner@apache.org
Precedence: bulk

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!