Re: [matplotlib-devel] cbook.py, maxdict class key checking on __setitem__

2009-09-18 Thread Eric Firing
Ryanitus wrote: > I may have found a bug in the __setitem__ method of the maxdict class. > Fixed--thanks for pointing it out. Eric > Since a dictionary is a mapping class, if an item is set that already > exists, it overwrites the previous. However, you are still appending that > item to _kill

[matplotlib-devel] cbook.py, maxdict class key checking on __setitem__

2009-08-21 Thread Ryanitus
I may have found a bug in the __setitem__ method of the maxdict class. Since a dictionary is a mapping class, if an item is set that already exists, it overwrites the previous. However, you are still appending that item to _killkeys regardless. In the case where 2 items with the same key were a