Terry J. Reedy added the comment: Converting tabs to spaces and deleting trailing whitespace are separate issues and should be discussed and possibly done separately.
I think trailing whitespace should be uniformly fixed and prohibited for C files as it seems to be for most other files, for reasons including the issue cited by Mark. I seems silly that one should be faced with either reverting changes by a mandated fixup program or pushing a somewhat messy patch. Does the repository check and reject new and changed lines with trailing whitespace, as it does for other file types? If so, then existing lines with whitespace are a trap for someone who makes a small change (such as '<=' to '<') and forgets to run patchcheck. I think generated code should also be generated without whitespace and I would also make this a separate patch. The review mindset for substantive code change is different from review of changes that must not make a substantive change. ---------- nosy: +terry.reedy _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue28145> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com