https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6645
John Hardin <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #29 from John Hardin <[email protected]> 2011-08-16 19:56:06 UTC --- (In reply to comment #28) > (In reply to comment #21) > > if (/^\S+ (\(.{0,100}\) ){0,1}by \S+ \(.{0,100}\) with qmail-scanner/) > > FWIW, rather than the {0,1} range quantifier, the equivalent "optional" > quantifier should be used. It's widely used and immediately obvious what it > does. > > (\(.{0,100}\) )? Also: While I see a \0 in the code following that RE, I don't see a \1 or $1 anywhere. Is a capturing subgroup really needed there? /^\S+ (?:\(.{0,100}\) )?by \S+ \(.{0,100}\) with qmail-scanner/ -- Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.
