This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eric.snow
Recipients Arfrever, Jim.Jewett, alex, asvetlov, benjamin.peterson, eric.araujo, eric.smith, eric.snow, ezio.melotti, flox, gregory.p.smith, introom, josh.r, mrabarnett, ncoghlan, ned.deily, pitrou, refi64, rhettinger, scoder, serhiy.storchaka, tonn81, westurner, yselivanov
Date 2015-05-25.15:46:19
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1432568780.15.0.578011021206.issue16991@psf.upfronthosting.co.za>
In-reply-to
Content
@mrab

gah! I could swear I originally had the _odict_clear_node first and had switched them due to a segfault.  It even crossed my mind on Friday but I didn't pursue it.  I'm guessing I did put the _odict_clear_node call first at some point but lost that fix along the way. :(  That's the "benefit" of having a patch languish for so long; you end up working on it from multiple hosts and sometimes lose bits and pieces.  Unfortunately only recently did I think to create a feature repo on which to track the on-going work.

Anyway, thanks for helping with the investigation.  The patch should be just about ready to commit at this point. :)  I'm going to give it a once over, check for any lingering ref leaks, and double-check with Raymond.  So I'm hopeful we can land this in the next few days.
History
Date User Action Args
2015-05-25 15:46:20eric.snowsetrecipients: + eric.snow, rhettinger, gregory.p.smith, ncoghlan, pitrou, scoder, eric.smith, benjamin.peterson, ned.deily, ezio.melotti, eric.araujo, mrabarnett, Arfrever, alex, asvetlov, flox, Jim.Jewett, serhiy.storchaka, yselivanov, westurner, refi64, josh.r, tonn81, introom
2015-05-25 15:46:20eric.snowsetmessageid: <1432568780.15.0.578011021206.issue16991@psf.upfronthosting.co.za>
2015-05-25 15:46:20eric.snowlinkissue16991 messages
2015-05-25 15:46:19eric.snowcreate