[issue17917] use PyModule_AddIntMacro() instead of PyModule_AddIntConstant() when applicable

Charles-François Natali report at bugs.python.org
Tue May 21 11:53:19 CEST 2013


Charles-François Natali added the comment:

> I don't understand "This is a lot of code churn, but it touches code that is unlikely to be modified otherwise, so I guess it's ok.". What does it mean it's okay when it touches on code that's unlikely to be modified?

The problem with refactoring is that it can complicate further merges
between branches. In this case, the code is nearly never touched, so
it's unlikely to be an issue.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue17917>
_______________________________________


More information about the Python-bugs-list mailing list