➜

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 scoder
Recipients andrewvaughanj, eric.smith, matrixise, scoder, vstinner
Date 2021-04-13.10:00:00
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1618308000.7.0.190347751853.issue43816@roundup.psfhosted.org>
In-reply-to
Content 
The macros were moved to "Includes/cpython/", not to "Includes/internal/". That suggests to me that they should use "extern C", so that C++ code that wants to make use of CPython internals can use them.

Basically, the way I see it, *all* header files in "Includes/" and "Includes/cpython/" should work with C++ code and thus have an "extern C". Only the header files in "Includes/internal/" should not have them.

Regardless, I've removed the macro usage from Cython so that the current state doesn't hurt our users.
History
Date User Action Args
2021-04-13 10:00:00scodersetrecipients: + scoder, vstinner, eric.smith, matrixise, andrewvaughanj
2021-04-13 10:00:00scodersetmessageid: <1618308000.7.0.190347751853.issue43816@roundup.psfhosted.org>
2021-04-13 10:00:00scoderlinkissue43816 messages
2021-04-13 10:00:00scodercreate