[issue36039] Replace append loops with list comprehensions

Raymond Hettinger report at bugs.python.org
Wed Feb 20 12:05:51 EST 2019


Raymond Hettinger <raymond.hettinger at gmail.com> added the comment:

I concur with all the other -1 comments and will mark this a closed.

FWIW, we generally discourage sweeping across the library with minor rewrites.  Guido articulated a principle of "holistic refactoring" where we make code improvements while working on the module as a whole.  This helps makes sure that code changes are made in the context of a thorough understanding of what the module is trying to do.  This also helps us reduce maintenance-induced-bugs where minor code adjustments create new bugs that weren't there before.

Thank you for taking a look at the source and please continue to do so. If you find a specific case that is problematic, feel free to post that one particular case.  That said, a better use of time is to take one of the many open tracker issues, research it, and propose a fix.

----------
nosy: +rhettinger
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue36039>
_______________________________________


More information about the Python-bugs-list mailing list