Message263976
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. |
|
Date |
User |
Action |
Args |
2016-04-22 07:45:36 | martin.panter | set | recipients:
+ martin.panter, loewis, rhettinger, vstinner, ned.deily, serhiy.storchaka, xiang.zhang |
2016-04-22 07:45:36 | martin.panter | set | messageid: <1461311136.19.0.487553427712.issue26824@psf.upfronthosting.co.za> |
2016-04-22 07:45:36 | martin.panter | link | issue26824 messages |
2016-04-22 07:45:36 | martin.panter | create | |
|