Message 378002 - Python tracker

➜

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	vstinner
Recipients	gvanrossum, serhiy.storchaka, vstinner
Date	2020-10-05.08:38:22
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1601887103.23.0.21690245045.issue41936@roundup.psfhosted.org>
In-reply-to

Content
I failed to find users of these macros outside the CPython code base, so it sounds safe to remove it, especially if it's documented in What's New in Python 3.10. If someone reports the removal as a regression and describes a legit use case, we can reconsider to revert the removal. Until that, I'm fine with removing them. -- I ran grep on Cython and numpy code base: I cannot find "Py_ALLOW_RECURSION", "Py_END_ALLOW_RECURSION" or "recursion_critical". On GitHub, I only found copies of Include/ceval.h (I looked at the first 5 pages of result): https://github.com/search?l=C&q=Py_ALLOW_RECURSION&type=Code

I failed to find users of these macros outside the CPython code base, so it sounds safe to remove it, especially if it's documented in What's New in Python 3.10.

If someone reports the removal as a regression and describes a legit use case, we can reconsider to revert the removal. Until that, I'm fine with removing them.

--

I ran grep on Cython and numpy code base: I cannot find "Py_ALLOW_RECURSION", "Py_END_ALLOW_RECURSION" or "recursion_critical".

On GitHub, I only found copies of Include/ceval.h (I looked at the first 5 pages of result):
https://github.com/search?l=C&q=Py_ALLOW_RECURSION&type=Code

History
Date	User	Action	Args
2020-10-05 08:38:23	vstinner	set	recipients: + vstinner, gvanrossum, serhiy.storchaka
2020-10-05 08:38:23	vstinner	set	messageid: <1601887103.23.0.21690245045.issue41936@roundup.psfhosted.org>
2020-10-05 08:38:23	vstinner	link	issue41936 messages
2020-10-05 08:38:22	vstinner	create