fkoyer commented on PR #33:
URL: https://github.com/apache/spamassassin/pull/33#issuecomment-5036495430

   Do you have any benchmarks to show that this is faster? There are two things 
we have to loop over: the set of rules and the set of body lines. It shouldn't 
matter performance-wise which is the inner loop and which is the outer loop. 
The complexity is still O(rules × lines). Also I think it's better to 
shortcircuit with `last` rather than doing `if (!$done{RULE})` for every 
iteration. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to