https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6419
Kevin A. McGrail <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #4752|0 |1 is obsolete| | --- Comment #6 from Kevin A. McGrail <[email protected]> 2010-05-27 13:35:25 EDT --- Created an attachment (id=4761) --> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4761) VERY small change to remove an extra semi-colon (In reply to comment #4) > Even though I would have preferred the public function to stay in PMS, but > that > might just be me and weird in another way. Oh, and could you please do an > s/;;/;/ on the Util.pm hunk of the patch? ;) >From memory, PMS is a class and I needed a procedural call to use the same function uniformly throughout. This issue really came down to implementations to do the same functionality that were not kept in sync. re: double semi-colon, new patch attached. (In reply to comment #5) > Forgot to mention: I didn't run the code, just checked the diff -- based on > the > previous discussion on the dev list. No worries, I've been running in production now since the day of the patch so I stand behind it. -- Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.
