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]