Serhiy Storchaka added the comment: That is why I haven't pushed the patch. Thanks you for your LGTMs Victor and Inada, but this is not enough.
In general I think that it is better to use Py_RETURN_NONE than "Py_INCREF(Py_None); return Py_None;". Some replacements already was done in the past, and some currently reviewed patches include also such changes. This can distract attention from the main purpose of patches. I think that pushing all changes in one big commit will make less harm than add small changes in other patches here and there. Concinelle proved his reliability and it is easy to check these changes. I'll push this patch if Raymond or other more conservative core developers accept it (or selected parts of it). ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue28999> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com