On Thu, Jan 8, 2009 at 17:31, Benjamin Peterson <[email protected]> wrote: > On Thu, Jan 8, 2009 at 2:52 PM, Brett Cannon <[email protected]> wrote: >> On Thu, Jan 8, 2009 at 12:43, Nick Coghlan <[email protected]> wrote: >>> >>> Even if we do adopt such a rule, C patches posted to the tracker should >>> still try to avoid including pure whitespace changes though - leaving >>> the whitespace changes in the patch tends to lead to patches that look >>> like "remove function body, add different function body" when only a >>> couple of lines have actually had significant changes. >>> >> >> That's fine with me. Correcting whitespace can be considered a committer's >> job. > > Maybe a rule could be added to Tools/scripts/patchcheck.py?
To do what? Re-indent automatically? Or notify the person that there seems to be a need to re-indent some code? -Brett _______________________________________________ Python-Dev mailing list [email protected] http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
