Author serhiy.storchaka
Recipients serhiy.storchaka
Date 2012-08-03.15:40:14
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1344008415.84.0.0454996292543.issue15550@psf.upfronthosting.co.za>
In-reply-to
Content
My editors are configured to remove trailing spaces (this is useful for removing artifacts of indentation). The flip side of this is that my patches sometimes contain unrelated trailing spaces fixes.

Trailing spaces are not significant in any CPython source file, their presence I believe mistake. Easier once we remove all spaces and then prevent the appearance of new, than constantly face to unrelated changes. I'm not attaching a patch (it is too big, over 5 MB), anyone can create it by the following commands:

    hg status -cn | tr '\n' '\0' | xargs -0 sed -i -re 's/[ \t]+$//'

It would be good if the Mercurial would had hook, which automatically remove trailing spaces or prohibit to commit patches that contain trailing spaces.
History
Date User Action Args
2012-08-03 15:40:15serhiy.storchakasetrecipients: + serhiy.storchaka
2012-08-03 15:40:15serhiy.storchakasetmessageid: <1344008415.84.0.0454996292543.issue15550@psf.upfronthosting.co.za>
2012-08-03 15:40:15serhiy.storchakalinkissue15550 messages
2012-08-03 15:40:14serhiy.storchakacreate