[issue2246] itertools.groupby() leaks memory with circular reference

2008-10-20 Thread Martin v. Löwis
Martin v. Löwis <[EMAIL PROTECTED]> added the comment: Already backported in r61287. -- status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___ __

[issue2246] itertools.groupby() leaks memory with circular reference

2008-10-20 Thread Martin v. Löwis
Changes by Martin v. Löwis <[EMAIL PROTECTED]>: -- status: closed -> open ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-lis

[issue2246] itertools.groupby() leaks memory with circular reference

2008-10-20 Thread Martin v. Löwis
Martin v. Löwis <[EMAIL PROTECTED]> added the comment: Backport candidate -- nosy: +loewis versions: +Python 2.5.3 -Python 2.4, Python 2.5, Python 2.6, Python 3.0 ___ Python tracker <[EMAIL PROTECTED]> __

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Raymond Hettinger
Raymond Hettinger added the comment: r61286. Applied a patch substantially similar to Alexanders. Thanks for the test case and the report. -- resolution: -> fixed status: open -> closed __ Tracker <[EMAIL PROTECTED]>

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Paul Pogonyshev
Paul Pogonyshev added the comment: Damn, I wrote a patch too ;) -- nosy: +_doublep __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailin

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Alexander Belopolsky
Alexander Belopolsky added the comment: Oops. Here is my patch anyways. -- keywords: +patch Added file: http://bugs.python.org/file9625/groupby-leak.diff __ Tracker <[EMAIL PROTECTED]> __

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Raymond Hettinger
Raymond Hettinger added the comment: No need. I'm already working on adding GC to the grouper. -- assignee: -> rhettinger __ Tracker <[EMAIL PROTECTED]> __ __

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Alexander Belopolsky
Alexander Belopolsky added the comment: It looks like the problem is that the internal grouper object becomes a part of a cycle: keyfunc -> grouper(x) -> keyfunc(tgtkey), but its type does not support GC. I will try to come up with a patch. __ Tracker <[EMAIL PRO

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Armin Ronacher
Changes by Armin Ronacher: -- nosy: +aronacher __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Alexander Belopolsky
Alexander Belopolsky added the comment: With the following patch: === --- Lib/test/test_itertools.py (revision 61284) +++ Lib/test/test_itertools.py (working copy) @@ -707,6 +707,12 @@ a = [] self.makecycle(takew

[issue2246] itertools.groupby() leaks memory with circular reference

2008-03-06 Thread Jeroen Ruigrok van der Werven
New submission from Jeroen Ruigrok van der Werven: Quoting from my email to Raymond: In the Trac/Genshi community we've been tracking a bit obscure memory leak that causes us a lot of problems. Please see http://trac.edgewall.org/ticket/6614 and then http://genshi.edgewall.org/ticket/190 for b