[slf4j-dev] [Bug 69] BasicMarker.add and remove are broken.

bugzilla-daemon at pixie.qos.ch bugzilla-daemon at pixie.qos.ch
Mon Jan 21 18:16:15 CET 2008


http://bugzilla.slf4j.org/show_bug.cgi?id=69





------- Comment #1 from joern at huxhorn.de  2008-01-21 18:16 -------
Created an attachment (id=23)
 --> (http://bugzilla.slf4j.org/attachment.cgi?id=23&action=view)
Patch fixing both described problems. Includes testcases that fail before patch
and work afterwards.

It would probably be a good idea to implement hashCode and equals for
BasicMarker and use contains instead of the for-loop with == comparison. I
didn't do that to minimize the code change.


-- 
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the slf4j-dev mailing list