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

--- Comment #29 from Mark Martinec <[email protected]> 2010-01-18 04:06:45 
UTC ---
(In reply to comment #24)
> What is the exact patch to test "use bytes"?  My corpus has lots of Japanese
> mail and I could run masscheck.

The exact location is not critical. Something like this would do:

--- Message.pm~ 2010-01-18 12:55:21.000000000 +0100
+++ Message.pm  2010-01-18 12:55:45.000000000 +0100
@@ -46,4 +46,5 @@
 use warnings;
 use re 'taint';
+use bytes;

 use Mail::SpamAssassin;


> But given 48 minutes vs 50 minutes is this really worthwhile of a risk to do
> last minute before release?

The change has a little effect on run times of majority of message, and
hence on the average times. Seems that only few messages are affected,
but there the difference can be substantial, as reported earlier
(factors like 5.7, 6.7, 1.9, 2.9 were reported under this bug, which
can have a large impact when a bunch of such message comes in, each
taking minutes to process).

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