style: fix styling and checkstyle warnings

src/main/java/org/terasology/launcher/game/GameService.java

@@ -50,19 +50,20 @@ public class GameService extends Service<Boolean> {
 
     public GameService() {
         setExecutor(Executors.newSingleThreadExecutor(
-            new ThreadFactoryBuilder()
-                    .setNameFormat("GameService-%d")
-                    .setDaemon(true)
-                    .setUncaughtExceptionHandler(this::exceptionHandler)
-                    .build()
+                new ThreadFactoryBuilder()
+                        .setNameFormat("GameService-%d")
+                        .setDaemon(true)
+                        .setUncaughtExceptionHandler(this::exceptionHandler)
+                        .build()
         ));
     }
 
     /**
      * Start a new game process with these settings.
+     *
      * @param installation the directory under which we will find libs/Terasology.jar, also used as the process's
-     *     working directory
-     * @param settings supplies other settings relevant to configuring a process
+     *                     working directory
+     * @param settings     supplies other settings relevant to configuring a process
      */
     @SuppressWarnings("checkstyle:HiddenField")
     public void start(Installation installation, Settings settings) {
@@ -116,10 +117,10 @@ public void restart() {
      * This class's configuration fields <em>must</em> be set before this is called.
      *
      * @throws com.google.common.base.VerifyException when fields are unset
-     * @throws RuntimeException when required files in the game directory are missing or inaccessible
+     * @throws RuntimeException                       when required files in the game directory are missing or inaccessible
      */
     @Override
-    protected RunGameTask createTask() throws GameVersionNotSupportedException{
+    protected RunGameTask createTask() throws GameVersionNotSupportedException {
         verifyNotNull(settings);
 
         GameStarter starter;
@@ -135,13 +136,17 @@ protected RunGameTask createTask() throws GameVersionNotSupportedException{
         return new RunGameTask(starter);
     }
 
-    /** After a task completes, reset to ready for the next. */
+    /**
+     * After a task completes, reset to ready for the next.
+     */
     @Override
     protected void succeeded() {
         reset();  // Ready to go again!
     }
 
-    /** Checks to see if the failure left any exceptions behind, then resets to ready. */
+    /**
+     * Checks to see if the failure left any exceptions behind, then resets to ready.
+     */
     @Override
     protected void failed() {
         // "Uncaught" exceptions from javafx's Task are actually caught and kept in a property,

src/main/java/org/terasology/launcher/game/GameStarter.java

@@ -39,7 +39,8 @@ final class GameStarter implements Callable<Process> {
      * @param logLevel          the minimum level of log events Terasology will include on its output stream to us
      */
     GameStarter(Installation installation, Path gameDataDirectory, JavaHeapSize heapMin, JavaHeapSize heapMax,
-                List<String> javaParams, List<String> gameParams, Level logLevel) throws IOException, GameVersionNotSupportedException, UnsupportedPlatformException {
+                List<String> javaParams, List<String> gameParams, Level logLevel)
+            throws IOException, GameVersionNotSupportedException, UnsupportedPlatformException {
         Semver engineVersion = installation.getEngineVersion();
         var gamePath = installation.path;
 

src/main/java/org/terasology/launcher/platform/Platform.java

@@ -14,6 +14,9 @@
  */
 public final class Platform {
 
+    final private OS os;
+    final private Arch arch;
+
     //TODO(skaldarnar): I'm not fully settled on how to model the Platform. I thought splitting this up into two enums
     //  for OS and Architecture would help with more type-safe construction of these values, simplify comparison to
     //  select the right JRE for a game, etc.
@@ -28,17 +31,12 @@ public final class Platform {
     //      MAC_AARCH64
     //  The biggest drawback of being super-strict here is that development on non-supported platforms becomes
     //  impossible where it was just "not ideal" before.
-
-
     public static final Set<Platform> SUPPORTED_PLATFORMS = Sets.newHashSet(
             new Platform(OS.WINDOWS, Arch.X64),
             new Platform(OS.LINUX, Arch.X64),
             new Platform(OS.MAC, Arch.X64)
     );
 
-    final private OS os;
-    final private Arch arch;
-
     public Platform(OS os, Arch arch) {
         this.os = os;
         this.arch = arch;