Re: Lots of trailing spaces in the code...

2010-05-09 Thread Jason
Here is the styling page https://cwiki.apache.org/TS/coding-style.html, I'll try to clean this up with anchor'ed menus and code block table cells. On Sun, May 9, 2010 at 8:57 PM, John Plevyak wrote: > > I agree (re running the script). > > We should also have emacs and vim configurations. > I th

Re: Lots of trailing spaces in the code...

2010-05-09 Thread John Plevyak
I agree (re running the script). We should also have emacs and vim configurations. I think Leif had a emacs one somewhere and I wonder if it has that option set. john On 5/9/2010 1:26 AM, Mladen Turk wrote: > Hi, > > According to the style guide and 'best practice' > there should be no traili

Re: Lots of trailing spaces in the code...

2010-05-09 Thread Mladen Turk
On 05/09/2010 05:09 PM, Leif Hedstrom wrote: Maybe do the next to lines "conditional" on diff between $f and $f.tmp actually finding a change being made? Extremely minor though, and in reality, if there is no change, the "mv" would be harmless anyways. Sure. Something like if [ `grep -e '[[:s

Re: Lots of trailing spaces in the code...

2010-05-09 Thread Leif Hedstrom
On 05/09/2010 02:26 AM, Mladen Turk wrote: This can be easily fixed with a simple script: #!/bin/sh for f do case "$f" in * ) sed -e 's/[[:space:]]*$//g' $f > $f.tmp Maybe do the next to lines "conditional" on diff between $f and $f.tmp actually finding a change being made? Extrem

Lots of trailing spaces in the code...

2010-05-09 Thread Mladen Turk
Hi, According to the style guide and 'best practice' there should be no trailing spaces in the code. Any descent editor has 'Trim trailing spaces on save' option and modifying editor preferences just for editing the code could be a hassle, cause those formatting edits could be intermixed with re