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

[bugzilla-daemon at pixie.qos.ch]

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





------- Comment #4 from listid at qos.ch  2008-02-05 15:07 -------
(In reply to comment #3)
> I don't know if 10 is sufficient for an OutOfMemoryError so I'm not sure if it
> will really fail without the patch in place but this test isn't that
> good/important anyway.

Forcing an OutOfMemoryError is not a goal per se.

> I just didn't have a better idea how to test/verify this because contains
> wouldn't have been sufficient and the OutOfMemoryError was the reason I
> originally found this bug.

Sure, that makes sense.

> Perhaps a test like adding a marker twice, removing it once and asserting that
> it isn't contained anymore would be better...

That's the idea. Add the same marker 10 times and check that it was added only
once.


-- 
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.