Need reviewer on addition of whitespace normalizer script (original) (raw)

Joe Darcy joe.darcy at oracle.com
Fri Mar 2 00:37:01 UTC 2012

Looks fine Kelly; great to have this sort of script included with the JDK itself.

-Joe

On 3/1/2012 1:32 PM, Kelly O'Hair wrote:

Need reviewer. Adding the whitespace normalizer script as a convenience to the jdk developers.

6625113: Add the normalize and rmkw perl script to the openjdk repository or openjdk site? http://cr.openjdk.java.net/~ohair/openjdk8/normalizer-script/webrev/ Probably a little history is warranted here. This script was originally written to normalize the whitespace in the jdk7 sources as they entered the Mercurial repositories in "changeset 0". It's been modified since then very slightly. I can't recall who wrote it (please speak up if you know) but it has been a valuable tool and I've had this CR to add it to the make/scripts directory for some time. The SCCS keyword removed (rmkw) was less useful, and I decided that it did not deserve being added. Why whitespace normalization? This was decided a long time ago when we had a raft of complaints from people viewing the sources with different tools and getting different views based on the TABs and trailing blanks or trailing newlines. So we decided to normalize on no TABs, no trailing blanks on lines, and no more than one blank line at the end of the file. This script was used to do that normalization. -kto

More information about the build-dev mailing list