Message 238279 - 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.

In-reply-to
Author	vstinner
Recipients	Arfrever, Joshua.J.Cogliati, Vitor.de.Lima, gustavotemple, jrincayc, lbianc, python-dev, vstinner
Date	2015年03月17日.11:24:16
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1426591456.43.0.985957441894.issue23644@psf.upfronthosting.co.za>

Content
> For third-party code, pyatomic.h is only needed by PyThreadState_GET() in pystate.h. Maybe we should hide completly pyatomic.h. Currently, pyatomic.h is not really used if Py_LIMITED_API is defined. pystate_cplusplus.patch: disable completly pyatomic.h on C++, and modify pystate.h: PyThreadState_GET() is just an alias to PyThreadState_Get() with compiled with C++. IMO pystate_cplusplus.patch makes more sense. I don't understand the purpose of atomicfix.patch or pyatomic_cpp.patch, PyThreadState_GET() will not work with these patches.

Content

> For third-party code, pyatomic.h is only needed by PyThreadState_GET() in pystate.h. Maybe we should hide completly pyatomic.h. Currently, pyatomic.h is not really used if Py_LIMITED_API is defined.
pystate_cplusplus.patch: disable completly pyatomic.h on C++, and modify pystate.h: PyThreadState_GET() is just an alias to PyThreadState_Get() with compiled with C++.
IMO pystate_cplusplus.patch makes more sense.
I don't understand the purpose of atomicfix.patch or pyatomic_cpp.patch, PyThreadState_GET() will not work with these patches.

History
Date	User	Action	Args
2015年03月17日 11:24:16	vstinner	set	recipients: + vstinner, Arfrever, jrincayc, python-dev, Joshua.J.Cogliati, Vitor.de.Lima, gustavotemple, lbianc
2015年03月17日 11:24:16	vstinner	set	messageid: <1426591456.43.0.985957441894.issue23644@psf.upfronthosting.co.za>
2015年03月17日 11:24:16	vstinner	link	issue23644 messages
2015年03月17日 11:24:16	vstinner	create

homepage