Looks fine to me. Had to recall the vi settings that visualize spaces
and tabs but
it was worth it. Some places looks hilarious, like this one:
http://cr.openjdk.java.net/~kizune/tmp/extra_spaces.png
I mean - someone spent a lot of time creating this invisible art.
Good it is going to be gone.
/Alex
On 8/5/2020 6:46 PM, Philip Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8240487
Webrev: http://cr.openjdk.java.net/~prr/8240487/
In advance of the move to Project Skara/git it is desirable to clean
up whitespace in source files
that are not currently checked by jcheck so we can add these
extensions to jcheck at that time.
The fix is therefore to remove tabs and trailing spaces.
The 3rd party harfbuzz library has .cc and .hh files but there are no
current violations there
since I've cleaned those up when importing harfbuzz upgrades.
There is one JDK file that relates to those that inherited tabs that
is fixed.
But almost all the fixes are in Objective C .m and .mm files.
JDK has no examples of .mm but JavaFX does so I was looking just to be
sure.
And all but one of the .m violations are in the desktop module which
is mainly because
that is where all but 5 of the Objective-C files are.
The only non-desktop violator is
./jdk.hotspot.agent/macosx/native/libsaproc/MacosxDebuggerLocal.m
and that is included in this webrev and why I've included
serviceability-dev.
-phil.