bpo-29878: Add global instances of int for 0 and 1. - #852
Conversation
Sorry, something went wrong.
|
@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tiran, @ncoghlan and @benjaminp to be potential reviewers. |
Sorry, something went wrong.
mdickinson
left a comment
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
vstinner
left a comment
There was a problem hiding this comment.
I'm really happy to see all these code go away! It's painful to have to check for errors on obvious things like that numbers 0 and 1!
The overall change LGTM, but IMHO it would be safer to use Py_CLEAR() in PyLong_Fini(). I have propose a minor coding style change.
Sorry, something went wrong.
| return -1; | ||
| } | ||
| *pdiv = (PyLongObject*)_PyLong_Zero; | ||
| Py_INCREF(_PyLong_Zero); |
There was a problem hiding this comment.
Whenever possible, even if we are protected by the GIL, I prefer to have Py_INCREF before assignement, for consistency.
Sorry, something went wrong.
| needed, but Python must forget about the reference or multiple | ||
| reinitializations will fail. */ | ||
| Py_DECREF(_PyLong_One); | ||
| Py_DECREF(_PyLong_Zero); |
There was a problem hiding this comment.
Py_CLEAR() would be safer if PyLong_Init() is called again (Py_Initialized called multiple times), no?
Sorry, something went wrong.
| return NULL; | ||
| } | ||
| start = _PyLong_Zero; | ||
| Py_INCREF(start); |
There was a problem hiding this comment.
ditto, in general I prefer to start with INCREF.
Sorry, something went wrong.
For internal use only.