Pete Hanson wrote: > From PerMsgStatus.pm as distributed with SA 2.31 (line number 1879): > > m/^((....)*)/; $_=$&; $leftover=$'; > > This can be replaced with: > > m/^((?:....)*)(.*)/ ; $_ = $1 ; $leftover = $2 ; > > and possibly simplified further. As near as I can tell, this is the > only place where the deadly $&, $', and $` variables are used in SA > code, and should be eliminated for performance reasons. > > Of course, this doesn't mean that modules used by SA don't also > include $&, $', and $`, but as far as I can tell, upgrading all > modules to the latest versions eliminates any remaining instances of > these variables in code used by SA.
Thanks. Patched in CVS. Matt. ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Spamassassin-talk mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/spamassassin-talk
