STINNER Victor <vstin...@python.org> added the comment:

I propose to rename "cpython/initconfig.h" to "cpython/cpython_initconfig.h".

#include "cpython/initconfig.h"

would become:

#include "cpython/cpython_initconfig.h"

So it becomes possible to include a cpython_xxx.h header from another 
cpython_xxx.h header (as done py cpython/pystate.h which includes 
cpython/initconfig.h).

--

Maybe a simpler change is simply to remove #include "cpython/initconfig.h" from 
cpython/pystate.h. Currently, cpython/pystate.h is included indirectly by 
Python.h at line 137, whereas cpython/initconfig.h is included by Python.h at 
line 135 (two lines before).

C extensions must include "Python.h" first and when "Python.h" is used, 
pystate.h gets the cpython/initconfig.h definitions as expected.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue39026>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to