On Thu, Jan 8, 2009 at 7:39 PM, Brett Cannon <br...@python.org> wrote:
> On Thu, Jan 8, 2009 at 17:31, Benjamin Peterson <benja...@python.org> wrote:
>> On Thu, Jan 8, 2009 at 2:52 PM, Brett Cannon <br...@python.org> wrote:
>>> On Thu, Jan 8, 2009 at 12:43, Nick Coghlan <ncogh...@gmail.com> 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?

I was thinking about notifying the person that their indentation was
wrong or they had trailing whitespace. Fixing it is bonus. :)



-- 
Regards,
Benjamin
_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to