RFR: 6639: Adding spotless for the rest of the java code

[Henrik Dafgård]

On Wed, 27 Nov 2019 21:43:48 GMT, Marcus Hirt <hirt at openjdk.org> wrote:

> Note that you need to run with -Puitests to include the uitests for the spotless analysis.
> 
> ----------------
> 
> Commits:
>  - ca9cbb8b: Spotless for the uitests
>  - f00eae06: Enabling spotless for everything but uitests (coming next)
>  - 3ce3862f: 6639: Spotless for all Java projects - initial cleanup
> 
> Changes: https://git.openjdk.java.net/jmc/pull/8/files
>  Webrev: https://webrevs.openjdk.java.net/jmc/8/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/JMC-6639
>   Stats: 1406 lines in 289 files changed: 757 ins; 110 del; 539 mod
>   Patch: https://git.openjdk.java.net/jmc/pull/8.diff
>   Fetch: git fetch https://git.openjdk.java.net/jmc pull/8/head:pull/8

application/org.openjdk.jmc.flightrecorder.controlpanel.ui/src/main/java/org/openjdk/jmc/flightrecorder/controlpanel/ui/ControlPanel.java line 127:

> 126: 		IPath lastFolder = lastPathStr.isEmpty() ? IDESupportToolkit.getDefaultRecordingFolder()
> 127: 				: Path.fromOSString(lastPathStr).removeLastSegments(1);
> 128: 

I actually kind of like this one. Can spotless be configured to allow separating ternary operations like the old style? If they can't be on one line or three then maybe they should be expanded to if/else statements? Having it on just two lines makes it a little bit harder to see what's going on.

PR: https://git.openjdk.java.net/jmc/pull/8