On Sun, 19 Jun 2011, Albin Tonnerre wrote:
> On Sun, 19 Jun 2011 12:38 +0200, Vincent Torri wrote : >> >> Hey >> >> another idea about trailing whitespaces : what about adding a rule in the >> toplevel Makefile.am, named for example remove-ws, that call find with a >> sed expression. >> >> I'll add it to Evil and test it a bit, and if nobody complains, i'll add >> it to the EFL + e. > > While it is nice to get rid of trailing whitespaces, I think that actively > removing them periodically does more harm than good. In particular, it adds a > lot of noise in the history and makes it harder to compare different > revisions. it would better than what we have today : some files that are committed when one think about those whitespaces, hence plenty of commits. At least, with that method, all the files of an EFL would be cleaned up in one call. Vincent > > If we indeed want to get rid of them, I'd be much happier if the SVN server > simply rejected commits containing them (the same holds true for formatting - > it's much better to ask people to submit properly formatted patches rather > than > going on a reformatting spree every now and then) > > Cheers, > -- > Albin Tonnerre > > ------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
