This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author exe
Recipients exe, georg.brandl
Date 2008-12-24.14:26:06
SpamBayes Score 4.851755e-07
Marked as misclassified No
Message-id <1230128805.53.0.886805938361.issue4737@psf.upfronthosting.co.za>
In-reply-to
Content
Hello!

1) In http://docs.python.org/3.0/extending/extending.html we see:
=========
Note that PyMODINIT_FUNC declares the function as void return type
=========

But thats not true, it's defined as PyObject*. I think this is outdated 
info in documentation.



2) In Doc/includes/noddy*.c assumed that PyMODINIT_FUNC returns nothing 
but this wrong. If you try "python3.1 -c 'import noddy'" you will see 
segfault. This patch will fix this problem:
--- ./noddy.c       2008-12-24 17:09:36.424870738 +0300
+++ ./noddy.c   2008-12-24 17:18:01.524869143 +0300
@@ -52,4 +52,6 @@
 
     Py_INCREF(&noddy_NoddyType);
     PyModule_AddObject(m, "Noddy", (PyObject *)&noddy_NoddyType);
+
+    return m;
 }


Other noddy*.c files are also requiring this patch.
History
Date User Action Args
2008-12-24 14:26:45exesetrecipients: + exe, georg.brandl
2008-12-24 14:26:45exesetmessageid: <1230128805.53.0.886805938361.issue4737@psf.upfronthosting.co.za>
2008-12-24 14:26:08exelinkissue4737 messages
2008-12-24 14:26:07execreate