Message114226
Benjamin,
Thanks for the feedback. Since you only commented on the test case, may I assume that the fix itself looked good to you?
I will work on revising the test case based on your comments. Since I ran into the bug while working with the ABCs in the collections module, that biased my thinking when writing the test. :-)
I needed to define__len__ because it's an abstract method in the ABC I used in the test (collections.Sized). I found that overriding it again in a sub-sub-class and calling it were necessary to trigger all of the ABC machinery leading to the leak. |
|
Date |
User |
Action |
Args |
2010-08-18 14:42:07 | stutzbach | set | recipients:
+ stutzbach, amaury.forgeotdarc, pitrou, jackdied, ajaksu2, benjamin.peterson, flox, BreamoreBoy |
2010-08-18 14:42:06 | stutzbach | set | messageid: <1282142526.99.0.735417114968.issue2521@psf.upfronthosting.co.za> |
2010-08-18 14:42:01 | stutzbach | link | issue2521 messages |
2010-08-18 14:42:00 | stutzbach | create | |
|