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 martin.panter
Recipients loewis, martin.panter, ned.deily, rhettinger, serhiy.storchaka, vstinner, xiang.zhang
Date 2016-04-22.07:45:36
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1461311136.19.0.487553427712.issue26824@psf.upfronthosting.co.za>
In-reply-to
Content
Most of Xiang’s changes are to Py<class>_Check() macros. I would expect them to be called with a generic PyObject pointer, and they do seem to be documented as accepting a PyObject pointer. Py_TYPE() is a macro that uses an unconditional cast. In general, these kinds of macros can hide errors that the compiler may otherwise pick up. E.g. if you accidentally pass an integer, or pointer to a pointer, etc, to Py_TYPE(), I think you will only get a warning or run-time crash, rather than a compile-time error. So while I am not that experienced with the C API, I suspect the change could have negative consequences.
History
Date User Action Args
2016-04-22 07:45:36martin.pantersetrecipients: + martin.panter, loewis, rhettinger, vstinner, ned.deily, serhiy.storchaka, xiang.zhang
2016-04-22 07:45:36martin.pantersetmessageid: <1461311136.19.0.487553427712.issue26824@psf.upfronthosting.co.za>
2016-04-22 07:45:36martin.panterlinkissue26824 messages
2016-04-22 07:45:36martin.pantercreate