Cleanup

Author: vstinner

Doc/whatsnew/3.14.rst

@@ -1018,7 +1018,7 @@ New features
 
   (Contributed by Victor Stinner in :gh:`107954`.)
 
-* Add a new import and export API for Python :class:`int` objects:
+* Add a new import and export API for Python :class:`int` objects (:pep:`757`):
 
   * :c:func:`PyLong_GetNativeLayout`;
   * :c:func:`PyLong_Export`;

Lib/test/test_capi/test_long.py

@@ -751,7 +751,7 @@ def test_long_export(self):
                     pylong_export(value)
 
     def test_longwriter_create(self):
-        # Test PyLong_Import()
+        # Test PyLongWriter_Create()
         layout = _testcapi.get_pylong_layout()
         base = 2 ** layout['bits_per_digit']
 

Misc/NEWS.d/next/C_API/2024-07-03-17-26-53.gh-issue-102471.XpmKYk.rst

@@ -1,4 +1,4 @@
-Add a new import and export API for Python :class:`int` objects:
+Add a new import and export API for Python :class:`int` objects (:pep:`757`):
 
 * :c:func:`PyLong_GetNativeLayout`;
 * :c:func:`PyLong_Export`;

Modules/_testcapi/long.c

@@ -145,50 +145,28 @@ static PyObject *
 layout_to_dict(const PyLongLayout *layout)
 {
     PyObject *dict = PyDict_New();
-
     if (dict == NULL) {
         goto error;
     }
 
-    PyObject *value = PyLong_FromUnsignedLong(layout->bits_per_digit);
-    if (value == NULL) {
-        goto error;
-    }
-    int res = PyDict_SetItemString(dict, "bits_per_digit", value);
-    Py_DECREF(value);
-    if (res < 0) {
-        goto error;
-    }
-
-    value = PyLong_FromUnsignedLong(layout->digit_size);
-    if (value == NULL) {
-        goto error;
-    }
-    res = PyDict_SetItemString(dict, "digit_size", value);
-    Py_DECREF(value);
-    if (res < 0) {
-        goto error;
-    }
-
-    value = PyLong_FromLong(layout->digits_order);
-    if (value == NULL) {
-        goto error;
-    }
-    res = PyDict_SetItemString(dict, "digits_order", value);
-    Py_DECREF(value);
-    if (res < 0) {
-        goto error;
-    }
-
-    value = PyLong_FromLong(layout->digit_endianness);
-    if (value == NULL) {
-        goto error;
-    }
-    res = PyDict_SetItemString(dict, "digit_endianness", value);
-    Py_DECREF(value);
-    if (res < 0) {
-        goto error;
-    }
+#define SET_DICT(KEY, EXPR) \
+    do { \
+        PyObject *value = (EXPR); \
+        if (value == NULL) { \
+            goto error; \
+        } \
+        int res = PyDict_SetItemString(dict, KEY, value); \
+        Py_DECREF(value); \
+        if (res < 0) { \
+            goto error; \
+        } \
+    } while (0)
+
+    SET_DICT("bits_per_digit", PyLong_FromUnsignedLong(layout->bits_per_digit));
+    SET_DICT("digit_size", PyLong_FromUnsignedLong(layout->digit_size));
+    SET_DICT("digits_order", PyLong_FromLong(layout->digits_order));
+    SET_DICT("digit_endianness", PyLong_FromLong(layout->digit_endianness));
+#undef SET_DICT
 
     return dict;