http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5049





------- Additional Comments From [EMAIL PROTECTED]  2006-08-22 01:27 -------
I'm -0.5.  The RE being used is a bit confusing:

s/\s*(?:#.*)?//g

why not:

s/\s*\#.*$//

The first RE seems questionable, since for instance, it'll always match on every
string (though it may not actually do anything).

Also:

+    next unless $key;          # skip blank lines (that remain)

would then have to change to:

next if ($key =~ /^\s*$/);

or

next unless ($key =~ /\S/);



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to