-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

could we add something like that to build.xml ->   <target name="scriptfix">


Adam Heath sent the following on 3/13/2009 10:14 AM:
> How do we feel about whitespace patches to java code?  I've added "let
> java_space_errors=1" to my $HOME/.vimrc, so when I use vim, I see bad
> spaces highlighted in red.  The things that get highlighed are
> trailing space or tab, lines that *only* contain space, and mixed
> space/tab at the beginning of a line.
> 
> Fix this is semi-automatic.  Trailing space and only space are fixed with:
> 
> ==
> find -name '*.java' | xargs sed -i -e 's/\([^\s]\)\s\+$/\1/;s/^[\s]\+$//'
> ==
> 
> Finding files with mixed space/tab is done with:
> 
> ==
> (tab=$(printf '\t'); find -name '*.java' | xargs egrep
> "^( +$tab|$tab+ )" -l)
> ==
> 
> If given the ok, I'll start fixing this minor issue.  However, it
> could cause conflicts with anyone else who has uncommitted changes.
> 
> ps: The patch against framework/base to fix these issues is 260k in
> size, 981 insertions/deletions, and only deals with java files.
> 
> 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJups0rP3NbaWWqE4RAlF9AKC/sK86TloFTUbRa7UZ2G8z3PumCQCePHkq
+wAxCW1VoR1pgDvfoPggZEo=
=GF7S
-----END PGP SIGNATURE-----

Reply via email to