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

--- Comment #7 from Karsten Bräckelmann <[email protected]> 2011-10-31 
22:46:07 UTC ---
(In reply to comment #6)
> Revised patch to use sprintf with variable placeholder for delimiter

Did you actually read comment 4? ;)

All that M::SA::Util::am_running_on_windows() code and explicitly setting your
own custom $delimiter might not be necessary. Basically, replacing the 4
occurrences of \n with $/ should do the trick, and make Perl magically use the
appropriate "newline" sequence.

Since $/ isn't a custom var, it should even be fine to use directly as a
replacement for \n in the sprintf() string -- I'd be rather scared if it ever
contains a %.

-- 
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