Apply suggestions from code review

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com> Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
python · serhiy-storchaka · Apr 8, 2025 · Dec 31, 2024 · Jan 2, 2025 · Jan 2, 2025
commit 9878b37001a944fd2bd16e70268432c29c8cc972
diff --git a/Doc/c-api/arg.rst b/Doc/c-api/arg.rst
@@ -344,17 +344,17 @@ Other objects
    in *items*.  The C arguments must correspond to the individual format units in
    *items*.  Format units for sequences may be nested.
 
-   If *items* contains format units which store
+   If *items* contains format units which store a
    :ref:`borrowed buffer <c-arg-borrowed-buffer>` or
-   :term:`reference <borrowed reference>`
+   a :term:`borrowed reference`
    (``y``, ``y#``, ``s``, ``s#``, ``z``, ``z#``, ``S``, ``Y``, ``U``,
-   ``O`` or ``O!``) the object must be a Python tuple.
+   ``O`` or ``O!``), the object must be a Python tuple.
    The *converter* for the ``O&`` format unit in *items* must not store
-   borrowed buffer or reference.
+   a borrowed buffer or reference.
 
    .. deprecated:: next
       Non-tuple sequences are deprecated if *items* contains format units
-      which store borrowed buffer or reference.
+      which store a borrowed buffer or reference.
 
 It is possible to pass "long" integers (integers whose value exceeds the
 platform's :c:macro:`LONG_MAX`) however no proper range checking is done --- the

diff --git a/Doc/whatsnew/3.14.rst b/Doc/whatsnew/3.14.rst
@@ -1230,8 +1230,8 @@ Deprecated
 * Non-tuple sequences are deprecated as argument for the ``(items)``
   format unit in :c:func:`PyArg_ParseTuple` and other
   :ref:`argument parsing <arg-parsing>` functions if *items* contains
-  format units which store :ref:`borrowed buffer <c-arg-borrowed-buffer>` or
-  :term:`reference <borrowed reference>`.
+  format units which store a :ref:`borrowed buffer <c-arg-borrowed-buffer>`
+  or a :term:`borrowed reference`.
   (Contributed by Serhiy Storchaka in :gh:`50333`.)
 
 * The previously undocumented function :c:func:`PySequence_In` is :term:`soft deprecated`.

diff --git a/Lib/test/test_capi/test_getargs.py b/Lib/test/test_capi/test_getargs.py
@@ -590,7 +590,7 @@ def test_tuple(self):
         ret = getargs_tuple(1, (2, 3))
         self.assertEqual(ret, (1,2,3))
 
-        # make sure invalid tuple arguments are handled correctly
+        # make sure invalid sequence arguments are handled correctly
         class TestSeq:
             def __len__(self):
                 return 2

diff --git a/Misc/NEWS.d/next/C_API/2024-12-31-15-28-14.gh-issue-50333.KxQUXa.rst b/Misc/NEWS.d/next/C_API/2024-12-31-15-28-14.gh-issue-50333.KxQUXa.rst
@@ -1,5 +1,5 @@
 Non-tuple sequences are deprecated as argument for the ``(items)`` format
 unit in :c:func:`PyArg_ParseTuple` and other :ref:`argument parsing
 <arg-parsing>` functions if *items* contains format units which store
-:ref:`borrowed buffer <c-arg-borrowed-buffer>` or :term:`reference <borrowed
-reference>`.
+a :ref:`borrowed buffer <c-arg-borrowed-buffer>` or a :term:`borrowed
+reference`.