Apply suggestions from code review

Co-authored-by: Petr Viktorin <[email protected]>
Co-authored-by: Kumar Aditya <[email protected]>
Co-authored-by: Victor Stinner <[email protected]>

Author: 4 people

Doc/c-api/import.rst

@@ -346,7 +346,7 @@ Importing Modules
    the same as for :c:func:`PyImport_AppendInittab`.
 
    On success, create and return a module object.
-   This module will not be initialized; call :c:func:`PyModule_Exec`
+   This module will not be initialized; call :c:func:`!PyModule_Exec`
    to initialize it.
    (Custom importers should do this in their
    :py:meth:`~importlib.abc.Loader.exec_module` method.)

Include/cpython/import.h

@@ -15,7 +15,7 @@ PyAPI_FUNC(int) PyImport_ExtendInittab(struct _inittab *newtab);
 // without needing to go through inittab
 PyAPI_FUNC(PyObject *) PyImport_CreateModuleFromInitfunc(
     PyObject *spec,
-    PyObject* (*initfunc)(void));
+    PyObject *(*initfunc)(void));
 
 struct _frozen {
     const char *name;                 /* ASCII encoded string */

Misc/NEWS.d/next/C_API/2025-11-08-10-51-50.gh-issue-116146.pCmx6L.rst

@@ -1,2 +1,2 @@
 Add a new :c:func:`PyImport_CreateModuleFromInitfunc` C-API for creating a
-module from a **spec** and **initfunc**. Patch by Itamar Oren.
+module from a *spec* and *initfunc*. Patch by Itamar Oren.

Programs/_testembed.c

@@ -166,7 +166,7 @@ static PyModuleDef embedded_ext = {
 static PyObject*
 PyInit_embedded_ext(void)
 {
-    // keep this a single-phase initialization module;
+    // keep this as a single-phase initialization module;
     // see test_create_module_from_initfunc
     return PyModule_Create(&embedded_ext);
 }

Python/import.c

@@ -2445,7 +2445,7 @@ create_builtin(
 
 PyObject*
 PyImport_CreateModuleFromInitfunc(
-    PyObject *spec, PyObject* (*initfunc)(void))
+    PyObject *spec, PyObject *(*initfunc)(void))
 {
     if (initfunc == NULL) {
         PyErr_BadInternalCall();