diff has options to deal with whitespace. -b --ignore-space-change -w --ignore-all-space
Tim Daly On Tue, 2011-10-25 at 07:35 +0200, Florent Hivert wrote: > On Mon, Oct 24, 2011 at 11:20:47AM -0700, Keshav Kini wrote: > > Hmm. I have to say I'm surprised that people are advocating for using > > trailing whitespace at all, as it seemed to me to be quite universally > > reviled in the programming world. But on some reflection, the only real > > problem with trailing whitespace - other than that it bloats files and that > > it, er, grates on certain types of minds :) - is that people remove it > > from, > > or, though I imagine this is rare, add it to otherwise untouched lines. I > > guess if hypothetically we had stuck to a "no trailing whitespace policy" > > from day 1 of Sage development, such a policy would have minimized diff > > cruft. But Sage's codebase is already so big that perhaps the best policy > > is > > just "don't purposelessly add or remove whitespace to lines that you aren't > > otherwise editing". As far as that goes, Jason's .emacs snippet seems > > useful > > for the developer's guide, indeed. > > +1 to this policy ! WE are trying to do that in the sage-combinat project. > > Systematically remove all trailing whitespaces creates a lot of conflicts in > the patch queue so I'm strongly against it. > > Cheers, > > Florent > -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
