Condense the implementation

Author: AA-Turner

Lib/test/test_sys.py

@@ -741,17 +741,15 @@ def test_thread_info(self):
 
     def test_abi_info(self):
         info = sys.abi_info
-        self.assertEqual(len(info.__dict__), 4)
+        info_keys = ['pointer_bits', 'free_threaded', 'debug', 'byteorder']
+        self.assertEqual(list(vars(info)), info_keys)
         pointer_bits = 64 if sys.maxsize > 2**32 else 32
         self.assertEqual(info.pointer_bits, pointer_bits)
+        self.assertEqual(info.free_threaded,
+                         bool(sysconfig.get_config_var('Py_GIL_DISABLED')))
+        self.assertEqual(info.debug,
+                         bool(sysconfig.get_config_var('Py_DEBUG')))
         self.assertEqual(info.byteorder, sys.byteorder)
-        for attr, flag in [
-            ("free_threaded", "Py_GIL_DISABLED"),
-            ("debug", "Py_DEBUG"),
-        ]:
-            self.assertEqual(getattr(info, attr, None),
-                             bool(sysconfig.get_config_var(flag)),
-                             f"for {attr}")
 
     @unittest.skipUnless(support.is_emscripten, "only available on Emscripten")
     def test_emscripten_info(self):

Misc/NEWS.d/next/Core_and_Builtins/2025-08-06-16-55-44.gh-issue-133143.l7CI9v.rst

@@ -1 +1 @@
-Add ``sys.abi_info`` object to make ABI information more easily accessible.
+Add :data:`sys.abi_info` object to make ABI information more easily accessible.

Python/sysmodule.c

@@ -3643,33 +3643,30 @@ make_impl_info(PyObject *version_info)
 static PyObject *
 make_abi_info(void)
 {
-    // New entries should be added when needed for a supported platform, or (for
-    // enabling an unsupported one) by core dev consensus.  Entries should be removed
-    // following PEP 387.
-    int res;
-    PyObject *abi_info, *value, *ns;
-    abi_info = PyDict_New();
+    // New entries should be added when needed for a supported platform,
+    // or by core dev consensus for enabling an unsupported one.
+
+    PyObject *abi_info = PyDict_New();
     if (abi_info == NULL) {
-        goto error;
+        return NULL;
     }
 
+    PyObject *value;
     value = PyLong_FromLong(sizeof(void *) * 8);
     if (value == NULL) {
         goto error;
     }
-    res = PyDict_SetItemString(abi_info, "pointer_bits", value);
-    Py_DECREF(value);
-    if (res < 0) {
+    if (PyDict_SetItemString(abi_info, "pointer_bits", value) < 0) {
         goto error;
     }
+    Py_DECREF(value);
 
 #ifdef Py_GIL_DISABLED
     value = Py_True;
 #else
     value = Py_False;
 #endif
-    res = PyDict_SetItemString(abi_info, "free_threaded", value);
-    if (res < 0) {
+    if (PyDict_SetItemString(abi_info, "free_threaded", value) < 0) {
         goto error;
     }
 
@@ -3678,31 +3675,26 @@ make_abi_info(void)
 #else
     value = Py_False;
 #endif
-    res = PyDict_SetItemString(abi_info, "debug", value);
-    if (res < 0) {
+    if (PyDict_SetItemString(abi_info, "debug", value) < 0) {
         goto error;
     }
 
 #if PY_BIG_ENDIAN
-    value = PyUnicode_FromString("big");
+    value = &_Py_ID(big);
 #else
-    value = PyUnicode_FromString("little");
+    value = &_Py_ID(little);
 #endif
-    if (value == NULL) {
-        goto error;
-    }
-    res = PyDict_SetItemString(abi_info, "byteorder", value);
-    Py_DECREF(value);
-    if (res < 0) {
+    if (PyDict_SetItemString(abi_info, "byteorder", value) < 0) {
         goto error;
     }
 
-    ns = _PyNamespace_New(abi_info);
+    PyObject *ns = _PyNamespace_New(abi_info);
     Py_DECREF(abi_info);
     return ns;
 
 error:
     Py_DECREF(abi_info);
+    Py_XDECREF(value);
     return NULL;
 }