httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Sutton <p...@c2.net>
Subject Re: cvs commit: apache-1.3/src/helpers UpdateHide
Date Wed, 04 Mar 1998 14:17:38 GMT
On Wed, 4 Mar 1998, Rodent of Unusual Size wrote:
> Ralf S. Engelschall wrote:
> > 
> >   -        next if ($name =~ m:^ap(_)|(api):);
> >   +        next if ($name =~ m/^ap(?:_|api)/);
> 
> I'm not going to veto, but I *am* going to complain: I find
> this an UNimprovement.  IMO it makes things harder to read
> rather than easier.  What's your point?

Er, pardon? Ralf's code works, the original regex doesn't. There shouldn't
even be a question of a complaint here, let alone a veto. For what its
worth, -1 on the original regex (it doesn't work), +1 for Ralf's. 

You could argue about perl4 compatibility, though (?: isn't in perl4, is
it?). 

//pcs


Mime
View raw message