[issue8912] `make patchcheck` should check the whitespace of .c/.h files

[Éric Araujo]

Éric Araujo <merwok at netwok.org> added the comment:

1) Okay, so I’ll refactor some code in patchcheck into a grep-like function to reduce duplication and support adding new checks.

2) I’ll add more useful report. Perhaps I’ll ask first on python-dev to see if I should add a command-line option or just improve it. Also, whether I should use logging or just output file names and lines.

3) Well, if your editor strips trailing spaces and adds final newlines, you won’t ever see make patchcheck complain, will you? Since the report will be helpful only to people with less smart editors, I think I’ll go with replacing instead of just reporting. Tabs will not be replaced, though, just reported.

4) See attached patch for py3k (no idea if this will go into 2.7 too).

----------
Added file: http://bugs.python.org/file17741/refactor.diff

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