The IntelliJ 7 default is to strip whitespace from modified lines only. So Im with Adam on this one. What editor do you use Vincent?
On Fri, Mar 6, 2009 at 9:20 AM, Adam Winer <[email protected]> wrote: > On Fri, Mar 6, 2009 at 8:52 AM, Vincent Siveton <[email protected]> > wrote: > > 2009/3/6 Adam Winer <[email protected]>: > >> What is the advantage of removing trailing whitespace? I can't see a > >> reason to do it at all. Unlike removing unnecessary imports (say), it > >> doesn't improve the codebase in any objective manner. > > > > According the given stats, it seems that the majority devs already > > have configured their favourite IDE to do it with pros/cons. > > What stats? As best I can tell, the default Eclipse install (what I'm > using for my Shindig work) doesn't do it, and it creates leading > whitespace for blank lines, which your checkins then remove. > > > [1] doesn't detail this case and I just want to be sure that the team > > shares the same configuration and remove or not trailing whitespace is > > one of them. So, include it to the Java style or not? > > My vote is for "not". > > Cheers, > Adam > > > > > Cheers, > > > > Vincent > > > > [1] http://cwiki.apache.org/SHINDIGxSITE/java-style.html > > >
