On Fri, 24 May 2002, Michael Moncur wrote:

> header ADVERT_CODE              Subject =~ /(^\s*|\s+)ADV:/i
> 
> and my suggested modification:
> 
> header ADVERT_CODE              Subject =~ /(?:^\s*|\s+|\[)(?:ADV|cc)[:\]]/i

On Fri, 24 May 2002, Matthew Cline wrote:

> That can be simplified a bit:
> 
>   /^\s*\[?(?:ADV|cc)[:\]]/i

That's not equivalent.  Note the placement of the carat WRT the first
alternation in the original rule. It should be:

        /(?:^\s*|\s+)\[?(?:ADV|cc)[:\]]/i

However ...

On Fri, 24 May 2002, Michael Moncur wrote:

> This would also catch the "cc:" that many spam subjects start with. I see
> more of these than ADV: these days.

I don't think the cc: has anything to do with advert codes?  How often do
you see [cc] ?  I suspect that should be a separate rule.


_______________________________________________________________

Don't miss the 2002 Sprint PCS Application Developer's Conference
August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm

_______________________________________________
Spamassassin-talk mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/spamassassin-talk

Reply via email to