You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@spamassassin.apache.org by bu...@issues.apache.org on 2010/07/22 13:04:55 UTC

[Bug 6466] TVD_PH_BODY_ACCOUNTS_PRE rule has redundant component

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6466

Karsten Bräckelmann <gu...@rudersport.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID

--- Comment #1 from Karsten Bräckelmann <gu...@rudersport.de> 2010-07-22 07:04:51 EDT ---
Condensing the alternations in the relevant section of the RE:
  (?:record[a-z]*|...)\b

There you can see the /[a-z]*/ is not redundant. It allows for more alpha chars
following the leading 'record' string, before the final end-of-word boundary.
Effectively meaning "optional".

Closing, since the stated issue is not a bug.

John, why did you have a close look at the rule in the first place? If the rule
caused FPs, please feel free to re-open this bug, giving more information about
the mail and SA rules hit.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.