Apply suggestions from code review

Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>

Author: skirpichev

Doc/whatsnew/3.15.rst

@@ -632,8 +632,8 @@ argparse
 array
 -----
 
-* Support the :c:expr:`float complex` and :c:expr:`double complex` C types
-  (formatting characters ``'F'`` and ``'D'`` respectively).
+* Support the :c:expr:`float complex` and :c:expr:`double complex` C types:
+  formatting characters ``'F'`` and ``'D'`` respectively.
   (Contributed by Sergey B Kirpichev in :gh:`146151`.)
 
 

Lib/test/test_array.py

@@ -1569,7 +1569,7 @@ def test_byteswap(self):
         if a.itemsize in (1, 2, 4, 8):
             b = array.array(self.typecode, self.example)
             b.byteswap()
-            if a.itemsize==1:
+            if a.itemsize == 1:
                 self.assertEqual(a, b)
             else:
                 # On alphas treating the byte swapped bit patterns as

Misc/NEWS.d/next/Library/2026-03-21-06-21-38.gh-issue-146151.yNpgml.rst

@@ -1,3 +1,3 @@
-Support the :c:expr:`float complex` and :c:expr:`double complex` C types in
-the :mod:`array` module (formatting characters ``'F'`` and ``'D'``
-respectively). Patch by Sergey B Kirpichev.
+Support the :c:expr:`float complex` and :c:expr:`double complex` C types
+in the :mod:`array` module: formatting characters ``'F'`` and ``'D'``
+respectively.  Patch by Sergey B Kirpichev.

Modules/arraymodule.c

@@ -1558,8 +1558,8 @@ array.array.byteswap
 
 Byteswap all items of the array.
 
-If the items in the array are not 1, 2, 4, 8 or 16 bytes in size, RuntimeError is
-raised.  Note, that for complex types the order of
+If the items in the array are not 1, 2, 4, 8 or 16 bytes in size, RuntimeError
+is raised.  Note, that for complex types the order of
 components (the real part, followed by imaginary part) is preserved.
 [clinic start generated code]*/
 
@@ -2359,8 +2359,9 @@ array__array_reconstructor_impl(PyObject *module, PyTypeObject *arraytype,
         const char *memstr = PyBytes_AS_STRING(items);
 
         converted_items = PyList_New(itemcount);
-        if (converted_items == NULL)
+        if (converted_items == NULL) {
             return NULL;
+        }
         for (i = 0; i < itemcount; i++) {
             PyObject *pycomplex = PyComplex_FromDoubles(
                 PyFloat_Unpack4(&memstr[i * 8], le),
@@ -2381,8 +2382,9 @@ array__array_reconstructor_impl(PyObject *module, PyTypeObject *arraytype,
         const char *memstr = PyBytes_AS_STRING(items);
 
         converted_items = PyList_New(itemcount);
-        if (converted_items == NULL)
+        if (converted_items == NULL) {
             return NULL;
+        }
         for (i = 0; i < itemcount; i++) {
             PyObject *pycomplex = PyComplex_FromDoubles(
                 PyFloat_Unpack8(&memstr[i * 16], le),