From 214562ed4ddc248b007f718ed92ebcc0c3669611 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hugo=20Beauz=C3=A9e-Luyssen?= Date: Sun, 2 Mar 2025 10:56:49 +0100 Subject: gh-130740: Move some `stdbool.h` includes after `Python.h` (#130738) Move some `#include ` after `#include "Python.h"` when `pyconfig.h` is not included first and when we are in a platform-agnostic context. This is to avoid having features defined by `stdbool.h` before those decided by `Python.h`. --- .../NEWS.d/next/Build/2025-03-01-18-27-42.gh-issue-130740.nDFSHR.rst | 2 ++ Modules/_hashopenssl.c | 2 +- Parser/string_parser.c | 4 ++-- Python/codegen.c | 4 ++-- Python/flowgraph.c | 5 ++--- Python/index_pool.c | 4 ++-- Python/instruction_sequence.c | 4 ++-- 7 files changed, 13 insertions(+), 12 deletions(-) create mode 100644 Misc/NEWS.d/next/Build/2025-03-01-18-27-42.gh-issue-130740.nDFSHR.rst diff --git a/Misc/NEWS.d/next/Build/2025-03-01-18-27-42.gh-issue-130740.nDFSHR.rst b/Misc/NEWS.d/next/Build/2025-03-01-18-27-42.gh-issue-130740.nDFSHR.rst new file mode 100644 index 0000000..61d416c --- /dev/null +++ b/Misc/NEWS.d/next/Build/2025-03-01-18-27-42.gh-issue-130740.nDFSHR.rst @@ -0,0 +1,2 @@ +Ensure that ``Python.h`` is included before ``stdbool.h`` unless ``pyconfig.h`` +is included before or in some platform-specific contexts. diff --git a/Modules/_hashopenssl.c b/Modules/_hashopenssl.c index 3f1deb8..f66b182 100644 --- a/Modules/_hashopenssl.c +++ b/Modules/_hashopenssl.c @@ -22,7 +22,6 @@ # define Py_BUILD_CORE_MODULE 1 #endif -#include #include "Python.h" #include "pycore_hashtable.h" #include "pycore_strhex.h" // _Py_strhex() @@ -37,6 +36,7 @@ #include #include +#include #ifndef OPENSSL_THREADS # error "OPENSSL_THREADS is not defined, Python requires thread-safe OpenSSL" diff --git a/Parser/string_parser.c b/Parser/string_parser.c index b93300b..9b557e9 100644 --- a/Parser/string_parser.c +++ b/Parser/string_parser.c @@ -1,5 +1,3 @@ -#include - #include #include "pycore_bytesobject.h" // _PyBytes_DecodeEscape() #include "pycore_unicodeobject.h" // _PyUnicode_DecodeUnicodeEscapeInternal() @@ -8,6 +6,8 @@ #include "pegen.h" #include "string_parser.h" +#include + //// STRING HANDLING FUNCTIONS //// static int diff --git a/Python/codegen.c b/Python/codegen.c index ecad8c2..8f1a298 100644 --- a/Python/codegen.c +++ b/Python/codegen.c @@ -12,8 +12,6 @@ * objects. */ -#include - #include "Python.h" #include "opcode.h" #include "pycore_ast.h" // _PyAST_GetDocString() @@ -32,6 +30,8 @@ #include "pycore_opcode_metadata.h" // _PyOpcode_opcode_metadata, _PyOpcode_num_popped/pushed #undef NEED_OPCODE_METADATA +#include + #define COMP_GENEXP 0 #define COMP_LISTCOMP 1 #define COMP_SETCOMP 2 diff --git a/Python/flowgraph.c b/Python/flowgraph.c index c5bdf10..fb3c73a 100644 --- a/Python/flowgraph.c +++ b/Python/flowgraph.c @@ -1,6 +1,3 @@ - -#include - #include "Python.h" #include "opcode.h" #include "pycore_flowgraph.h" @@ -12,6 +9,8 @@ #include "pycore_opcode_utils.h" #include "pycore_opcode_metadata.h" // OPCODE_HAS_ARG, etc +#include + #undef SUCCESS #undef ERROR diff --git a/Python/index_pool.c b/Python/index_pool.c index 526eccf..007c81a 100644 --- a/Python/index_pool.c +++ b/Python/index_pool.c @@ -1,10 +1,10 @@ -#include - #include "Python.h" #include "pycore_index_pool.h" #include "pycore_lock.h" +#include + #ifdef Py_GIL_DISABLED static inline void diff --git a/Python/instruction_sequence.c b/Python/instruction_sequence.c index e2607e9..ed40c06 100644 --- a/Python/instruction_sequence.c +++ b/Python/instruction_sequence.c @@ -5,8 +5,6 @@ */ -#include - #include "Python.h" #include "pycore_compile.h" // _PyCompile_EnsureArrayLargeEnough @@ -22,6 +20,8 @@ typedef _Py_SourceLocation location; #include "clinic/instruction_sequence.c.h" +#include + #undef SUCCESS #undef ERROR #define SUCCESS 0 -- cgit v0.12