]> granicus.if.org Git - python/commitdiff
#5548: do return the new module from PyMODINIT_FUNC functions.
authorGeorg Brandl <georg@python.org>
Tue, 31 Mar 2009 15:52:41 +0000 (15:52 +0000)
committerGeorg Brandl <georg@python.org>
Tue, 31 Mar 2009 15:52:41 +0000 (15:52 +0000)
Doc/extending/extending.rst
Doc/extending/newtypes.rst

index b035d94f9d7ccf3a21f997fb2fd3b104eea2942c..d7f1357c5e10b029731deab6f4610884d81ef22e 100644 (file)
@@ -1266,12 +1266,13 @@ All that a client module must do in order to have access to the function
    {
        PyObject *m;
 
-       m = Py_InitModule("client", ClientMethods);
+       m = PyModule_Create(&clientmodule);
        if (m == NULL)
-           return;
+           return NULL;
        if (import_spam() < 0)
-           return;
+           return NULL;
        /* additional initialization can happen here */
+       return m;
    }
 
 The main disadvantage of this approach is that the file :file:`spammodule.h` is
index c20e96f58910deb6df50db3d0e470fe0fc5954fd..83c95f2f4b66295f53e3568a90fe7fda9885d8e1 100644 (file)
@@ -871,6 +871,7 @@ the module's :cfunc:`init` function. ::
 
        Py_INCREF(&ShoddyType);
        PyModule_AddObject(m, "Shoddy", (PyObject *) &ShoddyType);
+       return m;
    }
 
 Before calling :cfunc:`PyType_Ready`, the type structure must have the