bpo-40170: Convert PyIter_Check macro to a function

Doc/c-api/iter.rst

@@ -9,8 +9,8 @@ There are two functions specifically for working with iterators.
 
 .. c:function:: int PyIter_Check(PyObject *o)
 
-   Return true if the object *o* supports the iterator protocol.  This
-   function always succeeds.
+   Return non-zero if the object *o* supports the iterator protocol, and ``0``
+   otherwise.  This function always succeeds.
 
 
 .. c:function:: PyObject* PyIter_Next(PyObject *o)

Include/cpython/abstract.h

@@ -327,9 +327,7 @@ PyAPI_FUNC(void) PyBuffer_Release(Py_buffer *view);
 
 /* ==== Iterators ================================================ */
 
-#define PyIter_Check(obj) \
-    (Py_TYPE(obj)->tp_iternext != NULL && \
-     Py_TYPE(obj)->tp_iternext != &_PyObject_NextNotImplemented)
+PyAPI_FUNC(int) PyIter_Check(PyObject *);
 
 /* === Sequence protocol ================================================ */
 

Misc/NEWS.d/next/Core and Builtins/2021-02-15-15-06-43.bpo-40170.ZYeSii.rst

@@ -0,0 +1,3 @@
+:c:func:`PyIter_Check` macro is now a function, in order to hide implementation
+details. The macro accessed :c:member:`PyTypeObject.tp_iternext` directly.
+Patch by Erlend E. Aasland.

Objects/abstract.c

@@ -2732,12 +2732,12 @@ PyObject_GetIter(PyObject *o)
     }
 }
 
-#undef PyIter_Check
-
-int PyIter_Check(PyObject *obj)
+int
+PyIter_Check(PyObject *obj)
 {
-    return Py_TYPE(obj)->tp_iternext != NULL &&
-           Py_TYPE(obj)->tp_iternext != &_PyObject_NextNotImplemented;
+    PyTypeObject *tp = Py_TYPE(obj);
+    return tp->tp_iternext != NULL &&
+           tp->tp_iternext != &_PyObject_NextNotImplemented;
 }
 
 /* Return next item.