Apply review feedback from vstinner and markshannon

python · markshannon · Sep 15, 2022 · May 3, 2022 · May 3, 2022 · May 3, 2022
commit 89cb4b25271be5a0e5dd7b18e464e3dc72d4195a
diff --git a/Doc/c-api/function.rst b/Doc/c-api/function.rst
@@ -87,6 +87,7 @@ There are a few functions specific to Python functions.
 
    Set the vectorcall field of a given function object *func*
-   Set the vectorcall field of a given function object *func*
+   Set the vectorcall field of a given function object *func*.
-   Set the vectorcall field of a given function object *func*
+   Set the vectorcall field of a given function object *func*.
 
+   .. versionadded:: 3.12
 
 .. c:function:: PyObject* PyFunction_GetClosure(PyObject *op)
 

@@ -237,7 +237,9 @@ New Features
   (Contributed by Wenzel Jakob in :gh:`93012`.)
 
 * Add new function :c:func:`PyFunction_SetVectorcall` to the C API
-  which sets the vectorcall field of a given PyFunctionObject
+  which sets the vectorcall field of a given :c:type:`PyFunctionObject`
-  which sets the vectorcall field of a given :c:type:`PyFunctionObject`
+  which sets the vectorcall field of a given :c:type:`PyFunctionObject`.
-  which sets the vectorcall field of a given :c:type:`PyFunctionObject`
+  which sets the vectorcall field of a given :c:type:`PyFunctionObject`.
+  Warning: extensions using this API must preserve the behavior
+  of the unaltered function!
   (Contributed by Andrew Frost in :gh:`92257`.)
 
 Porting to Python 3.12

diff --git a/Lib/test/test_call.py b/Lib/test/test_call.py
@@ -673,7 +673,7 @@ def f(num): return num + 1
         function_setvectorcall(f)
         # make sure specializer is triggered by running > 50 times
         for _ in range(51):
-            self.assertIsNone(f(num))
+            self.assertEqual("overridden", f(num))
 
 class A:
     def method_two_args(self, x, y):

diff --git a/Misc/NEWS.d/next/C API/2022-05-03-19-35-37.gh-issue-92193.61VoFL.rst b/Misc/NEWS.d/next/C API/2022-05-03-19-35-37.gh-issue-92193.61VoFL.rst
@@ -1,6 +1,5 @@
-CPython extensions providing optimized execution of Python bytecode (like Cinder JIT and Pyjion)
-can benefit from being able to modify the vectorcall field on instances of PyFunctionObject to allow calling the optimized path (e.g. JIT-compiled) directly.
+Add new function :c:func:`PyFunction_SetVectorcall` to the C API
+which sets the vectorcall field of a given :c:type:`PyFunctionObject`
-which sets the vectorcall field of a given :c:type:`PyFunctionObject`
+which sets the vectorcall field of a given :c:type:`PyFunctionObject`.
-which sets the vectorcall field of a given :c:type:`PyFunctionObject`
+which sets the vectorcall field of a given :c:type:`PyFunctionObject`.
 
-This PR introduces an API call where vectorcall field can be modified like so:
-
-`void PyFunction_SetVectorcall(PyFunctionObject *func, vectorcallfunc vectorcall);`
+Warning: extensions using this API must preserve the behavior
+of the unaltered function!
diff --git a/Modules/_testcapimodule.c b/Modules/_testcapimodule.c
@@ -5359,7 +5359,7 @@ test_pyobject_vectorcall(PyObject *self, PyObject *args)
 static PyObject *
 override_vectorcall(
     PyObject *callable, PyObject *const *args, size_t nargsf, PyObject *kwnames) {
-    Py_RETURN_NONE;
+    return PyUnicode_FromString("overridden");
 }
 
 static PyObject *