Hi everyone,

As a followup to the CheckStyle commits I recently made, I wanted to fix
some of the most common style problems.  One that I'm sure everyone is
familiar with is trailing whitespace in files.  There are ~1600 offenses in
the existing code base for *.java files alone according to my most recent
Jenkins build.

I wanted to drive this number down so that we can focus on "real" problems
that CheckStyle is finding.  So, I wrote a script to kill the whitespace.
http://pastebin.com/Qf0FAyfa  It might not be the most efficient thing in
the world but it works for this purpose.

Running this script over all of shindig-project touched 500 files and
generated a ~750kb patch file that is >800,000 lines long.  My question to
all of you is whether or not this is something worth putting on the
reviews.apache.org site or if I should just commit it.

Thoughts?

-Stanton

Reply via email to