New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get rid of references to PyInt in Py3 sources #67072
Comments
There are several references to PyInt in outdated comments in .c and .h files: $ find * -name '*.[ch]' -exec egrep -n --color -- 'PyInt\b' '{}' +
Include/longobject.h:34:/* It may be useful in the future. I've added it in the PyInt -> PyLong
Modules/fcntlmodule.c:157: into either a large positive number (PyLong or PyInt on 64-bit
Modules/fcntlmodule.c:158: platforms) or a negative number on others (32-bit PyInt)
Modules/_json.c:813: PyInt, PyLong, or PyFloat.
Modules/itertoolsmodule.c:3869: assert(cnt != PY_SSIZE_T_MAX && long_cnt == NULL && long_step==PyInt(1));
Python/ceval.c:4606:/* Extract a slice index from a PyInt or PyLong or an object with the |
I doubt that just removing PyInt is correct. At least in longobject.h current comment looks correct. And in fcntlmodule.c and itertoolsmodule.c proposed changes look questionable. There are also mentions of PyInt_* in *.rst and *.py files. |
New changeset 2f0c1937d716 by Serhiy Storchaka in branch 'default': New changeset c8d08de14030 by Serhiy Storchaka in branch '3.4': New changeset c0b2dacef35c by Serhiy Storchaka in branch 'default': |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: