bpo-45434: bytearrayobject.h no longer includes <stdarg.h> (GH-28913)

vstinner · web-flow · commit c63623a0a689 · 2021-10-13T04:37:55.000+02:00
bytearrayobject.h and _lzmamodule.c don't use va_list and so don't
need to include &lt;stdarg.h&gt;.
diff --git a/Include/bytearrayobject.h b/Include/bytearrayobject.h
@@ -6,8 +6,6 @@
 extern "C" {
 #endif
 
-#include <stdarg.h>
-
 /* Type PyByteArrayObject represents a mutable array of bytes.
  * The Python API is that of a sequence;
  * the bytes are mapped to ints in [0, 256).
diff --git a/Include/bytesobject.h b/Include/bytesobject.h
@@ -7,7 +7,7 @@
 extern "C" {
 #endif
 
-#include <stdarg.h>
+#include <stdarg.h>               // va_list
 
 /*
 Type PyBytesObject represents a byte string.  An extra zero byte is
diff --git a/Include/modsupport.h b/Include/modsupport.h
@@ -7,7 +7,7 @@ extern "C" {
 
 /* Module support interface */
 
-#include <stdarg.h>
+#include <stdarg.h>               // va_list
 
 /* If PY_SSIZE_T_CLEAN is defined, each functions treats #-specifier
    to mean Py_ssize_t */
diff --git a/Include/unicodeobject.h b/Include/unicodeobject.h
@@ -1,7 +1,7 @@
 #ifndef Py_UNICODEOBJECT_H
 #define Py_UNICODEOBJECT_H
 
-#include <stdarg.h>
+#include <stdarg.h>               // va_list
 
 /*
 
diff --git a/Modules/_lzmamodule.c b/Modules/_lzmamodule.c
@@ -10,7 +10,6 @@
 #include "Python.h"
 #include "structmember.h"         // PyMemberDef
 
-#include <stdarg.h>
 #include <string.h>
 
 #include <lzma.h>
