Jesús Cea Avión <j...@jcea.es> added the comment:

"PyErr_NoMemory()" returns always NULL, but it is declared as returning 
"PyObject *".

Could we change "return PyErr_NoMemory();" to "PyErr_NoMemory(); return 
NULL;"?. Maybe with some comment... A cast would silence the compiler but could 
be unclear. What do you think?

This is not a problem in 3.2 because the function patched returns a "PyObject 
*".

----------

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

Reply via email to