RFR (M/XXS) cleanup more non-indent white space issues (8047156)

Daniel D. Daugherty daniel.daugherty at oracle.com
Wed Jun 18 01:12:39 UTC 2014


Greetings,

I forgot to update my list of files to clean for the latest version
of the cleanup bucket so I missed these seven files when I sent out
the review for:

     JDK-8046758 cleanup non-indent white space issues prior to Contended
                 Locking cleanup bucket
     https://bugs.openjdk.java.net/browse/JDK-8046758

I have the fix for the following bug ready for JDK9 RT_Baseline:

     JDK-8047156 cleanup more non-indent white space issues prior to
                 Contended Locking cleanup bucket
     https://bugs.openjdk.java.net/browse/JDK-8047156

This is both a (M)edium and e(X)tra e(X)tra (S)mall review! Here is the
URL for the (M)edium webrev:

http://cr.openjdk.java.net/~dcubed/8047156-webrev/0-jdk9-hs-rt/

Since these are white space fixes, I recommend the udiff links instead
of the frames links. Of course, some folks prefer the patch file itself
for white space fixes.

Here is the URL for the e(X)tra e(X)tra (S)mall webrev which was 
generated with a version of webrev that ignores all whitespace changes:

http://cr.openjdk.java.net/~dcubed/8047156-webrev/0-jdk9-hs-rt-no_ws/

Again, I recommend the udiff links since that's the fastest way to see
that webrev_no_ws found no non-white space changes. The patch file for
this webrev shows all the white space changes.

Thanks, in advance, for any comments, questions or suggestions.

At the end of the review cycle for this fix, I will attach the version 
of the do_space_filter.ksh script used to do the work to JDK-8047156.

Dan




More information about the hotspot-runtime-dev mailing list