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 alex, barry, docs@python, eric.snow, ethan.furman, mark.dickinson, ncoghlan, serhiy.storchaka, vstinner
Date 2013-08-21.09:33:16
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1377077596.58.0.317556542548.issue17576@psf.upfronthosting.co.za>
In-reply-to
Content
Yet some nitpicks.

Currently the code of _PyLong_FromNbInt() is inlined and the do_decref flag is used to prevent needless change refcounts of int objects (see also issue18797). In proposed patch common code is extracted into the _PyLong_FromNbInt() function and int objects increfed and decrefed. Doesn't it affect a performance? PyLong_As* functions used in arguments parsing in a lot of builtins and their .performance is important.

If the patch slowdowns PyLong_As* functions we perhaps should check PyLong_CheckExact() before calling _PyLong_FromNbInt() and use the do_decref flag.

In general the idea and the patch LGTM.
History
Date User Action Args
2013-08-21 09:33:16serhiy.storchakasetrecipients: + serhiy.storchaka, barry, mark.dickinson, ncoghlan, vstinner, alex, docs@python, ethan.furman, eric.snow
2013-08-21 09:33:16serhiy.storchakasetmessageid: <1377077596.58.0.317556542548.issue17576@psf.upfronthosting.co.za>
2013-08-21 09:33:16serhiy.storchakalinkissue17576 messages
2013-08-21 09:33:16serhiy.storchakacreate