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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #3419 is|0                           |1
           obsolete|                            |
Attachment #3443 is|0                           |1
           obsolete|                            |




------- Additional Comments From [EMAIL PROTECTED]  2006-04-09 00:51 -------
Created an attachment (id=3464)
 --> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=3464&action=view)
Patch against 3.1 branch


What do you think of this simpler patch?

>From comment #8: "in other words, just fix the exact spots where we break out
of the loop (through return or last)"

If we break out of the loop with a return, there is no reason to use last. All
we have to do is insert the keys %hash before the return 1. The return 0
doesn't need it and we don't need a last.




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

Reply via email to