Author danielsh
Recipients Arfrever, danielsh, einarfd, eli.bendersky, ezio.melotti, flox, georg.brandl, jcea, larry, python-dev, santoso.wijaya, skrah
Date 2013-01-11.11:07:02
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <20130111110655.GG3556@lp-shahaf.local>
In-reply-to <1357893883.51.0.604044509271.issue16076@psf.upfronthosting.co.za>
Content
Ezio Melotti wrote on Fri, Jan 11, 2013 at 08:44:43 +0000:
> >>> ### one ref leaked for every child in __setstate__:
> >>> e2.__setstate__(p2)
> [76810 refs]
> >>> e2.__setstate__(p2)
> [76813 refs]
> >>> e2.__setstate__(p2)
> [76816 refs]
> 
> I'm not working on this anymore now, so someone more familiar with the
> code can take a look, see if my patch is correct, and fix the
> remaining leaks.
> 

Thank for narrowing it down so much.  Attached patch incorporates your
getstate fix and a fix for setstate.  It passes the -R 3:2 tests.
Files
File name Uploaded
memleak-v1.diff danielsh, 2013-01-11.11:07:01
History
Date User Action Args
2013-01-11 11:07:03danielshsetrecipients: + danielsh, georg.brandl, jcea, larry, ezio.melotti, Arfrever, eli.bendersky, skrah, flox, santoso.wijaya, python-dev, einarfd
2013-01-11 11:07:02danielshlinkissue16076 messages
2013-01-11 11:07:02danielshcreate