Author ncoghlan
Recipients Arfrever, barry, ncoghlan, pitrou
Date 2014-11-16.05:52:28
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1416117149.96.0.527880182935.issue22869@psf.upfronthosting.co.za>
In-reply-to
Content
It turns out *all* the global config variations are in pydebug.h (and they're basically the only thing in there, aside from the environment variable access macro). That seems a little weird to me (perhaps historical due to the early global config variables being debugging related?), but for now I've just added a couple of comments cross referencing pydebug.h from pylifecycle.c and vice-versa.

The latest version of the patch:

- moves the reference count & block allocation printing to object.[ch] as Antoine suggested (Macro = _PY_DEBUG_PRINT_TOTAL_REFS(); Symbol = _PyDebug_PrintTotalRefs())

- moves Py_OptimizeFlag to pylifecycle.c together with the other global configuration flags (and tweaks the order of definitions to exactly match the order of declarations in pydebug.h)

Since the extern declaration is actually there in pydebug.h, the reference to Py_InspectFlag from pythonrun.c is actually fine for now (although removing that direct reference will likely end up being part of the PEP 432 implementation).
History
Date User Action Args
2014-11-16 05:52:30ncoghlansetrecipients: + ncoghlan, barry, pitrou, Arfrever
2014-11-16 05:52:29ncoghlansetmessageid: <1416117149.96.0.527880182935.issue22869@psf.upfronthosting.co.za>
2014-11-16 05:52:29ncoghlanlinkissue22869 messages
2014-11-16 05:52:28ncoghlancreate