[issue35134] Move !Py_LIMITED_API to Include/pycapi/
Nick Coghlan
report at bugs.python.org
Mon Nov 5 06:53:31 EST 2018
Nick Coghlan <ncoghlan at gmail.com> added the comment:
On actually looking at the initial changes in the PR:
* declarations that aren't part of the stable ABI in any version (i.e. "#ifndef PY_LIMITED_API", "#if !defined(PY_LIMITED_API)") should move to the new directory
* declarations that are part of the stable ABI in *some* version should remain where they are (i.e. in "Include/*.h")
In your initial PR, the only API that subtle distinction affects is PyObject_Calloc (since that's a new addition to the stable ABI in 3.5+), and moving that back to the public header means you can add the desired "Py_LIMITED_API is not defined" check to the header in the new directory.
----------
_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue35134>
_______________________________________
More information about the Python-bugs-list
mailing list