bpo-40241: Add PyObject_GC_IsTracked and PyObject_GC_IsFinalized to the public C-API by pablogsal · Pull Request #19461 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.8k
Conversation
Member
3b361bc to
6a6d61c
Compare
vstinner
reviewed
Modules/gcmodule.c Outdated
Doc/c-api/gcsupport.rst Outdated
…ed to the public C-API
vstinner
approved these changes
vstinner
left a comment
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Fun fact: PyObject_GC_Track() is documented to accept "PyObject *", whereas it currently accepts "void *".
https://docs.python.org/dev/c-api/gcsupport.html#c.PyObject_GC_Track
| Py_SET_REFCNT(self, refcnt); | ||
| } | ||
| assert(!PyType_IS_GC(Py_TYPE(self)) || _PyObject_GC_IS_TRACKED(self)); | ||
| assert(!PyType_IS_GC(Py_TYPE(self)) || PyObject_GC_IsTracked(self)); |
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this change :-)
f13072b
into
python:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants
