Apply suggestions from code review

Author: XuehaiPan

Doc/library/sys.rst

@@ -31,8 +31,8 @@ always available. Unless explicitly noted otherwise, all variables are read-only
       means the :data:`sys.abiflags` member will always be available on all
       platforms starting from Python 3.16.
 
-      See :ref:`whatsnew314-sys-abiflags-change` in the *What's New* for more
-      details.
+      See the notes for :ref:`incoming change to sys.abiflags <whatsnew314-sys-abiflags-change>`
+      in the *What's New* for more details.
 
 
 .. function:: addaudithook(hook)

Doc/whatsnew/3.14.rst

@@ -932,6 +932,7 @@ sys
 * Schedule a change for availability of :data:`sys.abiflags` on Windows. A
   :exc:`DeprecationWarning` will be emitted if the :data:`sys.abiflags` member
   is accessed on Windows before Python 3.16.
+  (Contributed by Xuehai Pan in :gh:`131717`.)
 
 .. _whatsnew314-sys-abiflags-change:
 

Python/sysmodule.c

@@ -77,8 +77,8 @@ module sys
 #ifndef ABIFLAGS
 
 // XXX: remove this and related code after set sys.abiflags on Windows in 3.16.
-static int
-_warn_incoming_sys_abiflags_change()
+static inline int
+_warn_incoming_sys_abiflags_change(void)
 {
     return PyErr_WarnEx(
         PyExc_DeprecationWarning,
@@ -167,8 +167,7 @@ _PySys_GetOptionalAttr(PyObject *name, PyObject **value)
 #ifndef ABIFLAGS
     if (ret == 0 && _PyUnicode_EqualToASCIIString(name, "abiflags")) {
         if (_warn_incoming_sys_abiflags_change() < 0) {
-            Py_XDECREF(*value);
-            *value = NULL;
+            Py_CLEAR(*value);
             return -1;
         }
     }
@@ -189,8 +188,7 @@ _PySys_GetOptionalAttrString(const char *name, PyObject **value)
 #ifndef ABIFLAGS
     if (ret == 0 && strcmp(name, "abiflags") == 0) {
         if (_warn_incoming_sys_abiflags_change() < 0) {
-            Py_XDECREF(*value);
-            *value = NULL;
+            Py_CLEAR(*value);
             return -1;
         }
     }