procmail
[Top] [All Lists]

RE: procmail regex style using Or construct

2009-02-21 14:12:26
Ruud H.G. van Tol wrote on Friday, February 20, 2009 12:54 AM:

Professional Software Engineering wrote:

 * MATCH ?? ^^(1|2|3|4|5|6|7)^^

Alternative:

   * MATCH ?? ^^[1-7]^^

NOT a good way to do it

It's fine.

I agree with Ruud.  Sean, don't let your personal stylistic
preferences become so prejudicial that they devolve into
general warnings.  There's nothing at all wrong with the
syntax, and it's the one I prefer myself.  It's fine to
have a stylistic preference for yourself and to state your
reasons, but don't call good syntax bad because of that.

Also, we did hash out the relative efficiencies years ago
and came to the conclusion, iirc, that "[charclass]" is
the tiniest bit more efficient than "(c|h|a|r|{...})".
We're talking microscopic differences, of course, though,
and not something anybody in his right mind would need to
worry about in production.  (Which is why I worry about it.) :-)

Dallman

____________________________________________________________
procmail mailing list   Procmail homepage: http://www.procmail.org/
procmail(_at_)lists(_dot_)RWTH-Aachen(_dot_)DE
http://MailMan.RWTH-Aachen.DE/mailman/listinfo/procmail

<Prev in Thread] Current Thread [Next in Thread>