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

--- Comment #10 from Kevin A. McGrail <[email protected]> 2011-11-01 18:08:55 
UTC ---

>   $delimiter =~ s/[\r\n]//g;
> 
> That's backwards, you wanted to weed out any *non* [\r\n] chars.

Thanks.  I meant $delimiter =~ s/[^\r\n]//g;

> Pending confirmation of $/ doing the expected on Windows systems (Daniel?), 
> I'd
> prefer to drop that entire $delimiter dance, and simply use $/ instead of \n.

It's a fair argument but requires testing. 

> (As a side-note, the caveats in comment 4 and 5 about old Mac OS <= 9 appears
> to be a non-issue. \n is a logical newline, and evaluates appropriately even 
> on
> old Mac OS systems. See perlport, section Newline.)

I was worried ;-)

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

Reply via email to