Message244039
@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. |
|
Date |
User |
Action |
Args |
2015-05-25 15:46:20 | eric.snow | set | recipients:
+ 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:20 | eric.snow | set | messageid: <1432568780.15.0.578011021206.issue16991@psf.upfronthosting.co.za> |
2015-05-25 15:46:20 | eric.snow | link | issue16991 messages |
2015-05-25 15:46:19 | eric.snow | create | |
|