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





------- Additional Comments From [EMAIL PROTECTED]  2006-12-25 22:46 -------
(In reply to comment #9)
> Please don't beat me.... :-)

:)

> While Text::Wrap did change, the change uncovered an SA bug.

I disagree, please see below.

> The pattern '(?<=[\s,])' (Passed in the line:
> ./lib/Mail/SpamAssassin/PerMsgStatus.pm:996:      $hdr =
> Mail::SpamAssassin::Util::wrap($hdr, "\t", "", 79, 0, '(?<=[\s,])'); 
> 
> is used with a *.

Due to a change in Text::Wrap, yes.

> What may have been meant here was simply '[\s,]'.

Nope.  Originally that's what it was.  However, it caused bug 2165.  The
solution (r5605 if anyone's interested) was to use the zero width assertion
which avoided removing the break character.

Unless I'm missing something in the Text::Wrap docs, there's no other way to say
"break on this character, but don't remove it from the string".



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

Reply via email to