You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Includes/internal/pycore_code.h uses static_assert() but does not inlcude pymacro.h, should it?
AFAIU pymacro.h makes sure that static_assert() is correctly defined for all supported compilers and platforms. Not including it in Includes/internal/pycore_code.h implicitly relies on pymacro.h being included before or via transitive includes.
I've found this while investigating a Cython extension module build failure. Cython includes the private header and thus requires static_macro() to be defined.
Bug report
Bug description:
Includes/internal/pycore_code.husesstatic_assert()but does not inlcudepymacro.h, should it?AFAIU
pymacro.hmakes sure thatstatic_assert()is correctly defined for all supported compilers and platforms. Not including it inIncludes/internal/pycore_code.himplicitly relies onpymacro.hbeing included before or via transitive includes.I've found this while investigating a Cython extension module build failure. Cython includes the private header and thus requires
static_macro()to be defined.CPython versions tested on:
3.13, CPython main branch
Operating systems tested on:
No response
Linked PRs