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 serhiy.storchaka
Recipients eric.snow, python-dev, r.david.murray, rhettinger, serhiy.storchaka, vstinner
Date 2015-07-24.05:41:39
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1437716499.96.0.535548849968.issue24681@psf.upfronthosting.co.za>
In-reply-to
Content
The patch makes set_add_entry() inconsistent not only with dict, but with other set methods that use set_lookkey(). I don't know if there is some subtle bug here, but I feel myself slightly uncomfortable with it. Also the new code looks a little less readable to me. Current code is straightforward translation from Python to C (first check error, then check that the result still is actual, finally check the result itself), the new code forces you to stop and think, it's correctness is not so obvious.

Even if all good with the patch, please don't commit it right now. Wait at least a week and then commit. May be someone other will find a flaw in the patch.
History
Date User Action Args
2015-07-24 05:41:40serhiy.storchakasetrecipients: + serhiy.storchaka, rhettinger, vstinner, r.david.murray, python-dev, eric.snow
2015-07-24 05:41:39serhiy.storchakasetmessageid: <1437716499.96.0.535548849968.issue24681@psf.upfronthosting.co.za>
2015-07-24 05:41:39serhiy.storchakalinkissue24681 messages
2015-07-24 05:41:39serhiy.storchakacreate