Marc-Andre Lemburg added the comment:

On 10.12.2012 11:39, Robin Schreiber wrote:
> 
> Robin Schreiber added the comment:
> 
> I have updated the patch to work again with the current version of the 
> _datetimemodule. 

Please use "_Py_" prefixes for private symbols you put in the header
files, e.g. _datetimemodulestate and the macros.

Question: What happens if PyModule_GetState() or PyState_FindModule()
raise an exception and return NULL ?

The current code will segfault in such a situation.

Thanks,
-- 
Marc-Andre Lemburg
eGenix.com

----------
nosy: +lemburg

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue15390>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to