Author mark.dickinson
Recipients belopolsky, brett.cannon, eric.araujo, mark.dickinson, pitrou
Date 2010-06-22.11:21:17
SpamBayes Score 0.0103411
Marked as misclassified No
Message-id <1277205679.55.0.161164430556.issue8912@psf.upfronthosting.co.za>
In-reply-to
Content
> I think I’ll go with replacing instead of just reporting.

How about a --report-only (insert better name here) command line option to patchcheck, then?

My editor doesn't delete trailing whitespace *automatically*, but it's trivial to remove trailing whitespace from a file if you know it's there (M-x delete-trailing-whitespace, I believe);  some people (like me) might prefer to make all changes within the editor rather than having an external tool do it for them.  This is especially true if you've got the editor open for the relevant file at the time, since you can end up with two different conflicting versions of the file---one in the editor and one on disk;  saving in the editor can undo the changes that patchcheck made on disk.
History
Date User Action Args
2010-06-22 11:21:19mark.dickinsonsetrecipients: + mark.dickinson, brett.cannon, belopolsky, pitrou, eric.araujo
2010-06-22 11:21:19mark.dickinsonsetmessageid: <1277205679.55.0.161164430556.issue8912@psf.upfronthosting.co.za>
2010-06-22 11:21:17mark.dickinsonlinkissue8912 messages
2010-06-22 11:21:17mark.dickinsoncreate