pythongh-116869: Make C API compatible with ISO C90 (python#116950)

vstinner · web-flow · commit a9c304cf020e · 2024-03-18T20:16:58.000+01:00
Make the C API compatible with -Werror=declaration-after-statement
compiler flag again.
diff --git a/Include/cpython/longintrepr.h b/Include/cpython/longintrepr.h
@@ -129,9 +129,10 @@ _PyLong_IsCompact(const PyLongObject* op) {
 static inline Py_ssize_t
 _PyLong_CompactValue(const PyLongObject *op)
 {
+    Py_ssize_t sign;
     assert(PyType_HasFeature((op)->ob_base.ob_type, Py_TPFLAGS_LONG_SUBCLASS));
     assert(PyUnstable_Long_IsCompact(op));
-    Py_ssize_t sign = 1 - (op->long_value.lv_tag & _PyLong_SIGN_MASK);
+    sign = 1 - (op->long_value.lv_tag & _PyLong_SIGN_MASK);
     return sign * (Py_ssize_t)op->long_value.ob_digit[0];
 }
 
diff --git a/Include/object.h b/Include/object.h
@@ -343,8 +343,7 @@ PyAPI_DATA(PyTypeObject) PyBool_Type;
 static inline Py_ssize_t Py_SIZE(PyObject *ob) {
     assert(ob->ob_type != &PyLong_Type);
     assert(ob->ob_type != &PyBool_Type);
-    PyVarObject *var_ob = _PyVarObject_CAST(ob);
-    return var_ob->ob_size;
+    return  _PyVarObject_CAST(ob)->ob_size;
 }
 #if !defined(Py_LIMITED_API) || Py_LIMITED_API+0 < 0x030b0000
 #  define Py_SIZE(ob) Py_SIZE(_PyObject_CAST(ob))
diff --git a/Misc/NEWS.d/next/C API/2024-03-18-09-58-46.gh-issue-116869.LFDVKM.rst b/Misc/NEWS.d/next/C API/2024-03-18-09-58-46.gh-issue-116869.LFDVKM.rst
@@ -0,0 +1,2 @@
+Make the C API compatible with ``-Werror=declaration-after-statement``
+compiler flag again. Patch by Victor Stinner.

Original file line number	Diff line number	Diff line change
`@@ -129,9 +129,10 @@ _PyLong_IsCompact(const PyLongObject* op) {`
`129`	`129`	`static inline Py_ssize_t`
`130`	`130`	`_PyLong_CompactValue(const PyLongObject *op)`
`131`	`131`	`{`
	`132`	`+ Py_ssize_t sign;`
`132`	`133`	`assert(PyType_HasFeature((op)->ob_base.ob_type, Py_TPFLAGS_LONG_SUBCLASS));`
`133`	`134`	`assert(PyUnstable_Long_IsCompact(op));`
`134`		`- Py_ssize_t sign = 1 - (op->long_value.lv_tag & _PyLong_SIGN_MASK);`
	`135`	`+ sign = 1 - (op->long_value.lv_tag & _PyLong_SIGN_MASK);`
`135`	`136`	`return sign * (Py_ssize_t)op->long_value.ob_digit[0];`
`136`	`137`	`}`
`137`	`138`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+Make the C API compatible with ``-Werror=declaration-after-statement``
	`2`	`+compiler flag again. Patch by Victor Stinner.`