Blessed modifiers order

Author: alexeysemenyukoracle

src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxPackagingPipeline.java

@@ -81,6 +81,6 @@ private static void writeLauncherIcons(
         }
     }
 
-    final static LinuxApplicationLayout APPLICATION_LAYOUT = LinuxApplicationLayout.create(
+    static final LinuxApplicationLayout APPLICATION_LAYOUT = LinuxApplicationLayout.create(
             ApplicationLayoutUtils.PLATFORM_APPLICATION_LAYOUT, Path.of("lib/libapplauncher.so"));
 }

src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/Codesign.java

@@ -39,7 +39,7 @@
 
 public final class Codesign {
 
-    public final static class CodesignException extends Exception {
+    public static final class CodesignException extends Exception {
 
         CodesignException(String[] output) {
             this.output = output;
@@ -54,7 +54,7 @@ String[] getOutput() {
         private static final long serialVersionUID = 1L;
     }
 
-    public final static class Builder {
+    public static final class Builder {
 
         private Builder(Supplier<List<String>> args) {
             this.args = Objects.requireNonNull(args);

src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/Keychain.java

@@ -86,7 +86,7 @@ static List<Keychain> listKeychains() {
         }).map(Keychain::new).toList();
     }
 
-    final static class KeychainException extends RuntimeException {
+    static final class KeychainException extends RuntimeException {
 
         KeychainException(String msg) {
             super(msg);

src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacPackagingPipeline.java

@@ -425,6 +425,6 @@ public void execute(TaskAction taskAction) throws IOException, PackagerException
         }
     }
 
-    final static MacApplicationLayout APPLICATION_LAYOUT = MacApplicationLayout.create(
+    static final MacApplicationLayout APPLICATION_LAYOUT = MacApplicationLayout.create(
             ApplicationLayoutUtils.PLATFORM_APPLICATION_LAYOUT, Path.of("Contents/runtime"));
 }

src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/model/MacFileAssociation.java

@@ -28,7 +28,7 @@
 
 public interface MacFileAssociation extends FileAssociation, MacFileAssociationMixin {
 
-    static public MacFileAssociation create(FileAssociation fa, MacFileAssociationMixin mixin) {
+    public static MacFileAssociation create(FileAssociation fa, MacFileAssociationMixin mixin) {
         return CompositeProxy.create(MacFileAssociation.class, fa, mixin);
     }
 }

src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/model/MacPackage.java

@@ -46,5 +46,5 @@ public static MacPackage create(Package pkg, MacPackageMixin mixin) {
         return CompositeProxy.create(MacPackage.class, pkg, mixin);
     }
 
-    public final static RuntimeLayout RUNTIME_PACKAGE_LAYOUT = RuntimeLayout.create(Path.of("Contents/Home"));
+    public static final RuntimeLayout RUNTIME_PACKAGE_LAYOUT = RuntimeLayout.create(Path.of("Contents/Home"));
 }

src/jdk.jpackage/share/classes/jdk/jpackage/internal/AppImageFile.java

@@ -237,7 +237,7 @@ static String getPlatform() {
         return PLATFORM_LABELS.get(OperatingSystem.current());
     }
 
-    private final static class AppImageProperties {
+    private static final class AppImageProperties {
         private AppImageProperties(Map<String, String> data, Set<String> stdKeys) {
             this.data = data;
             this.stdKeys = stdKeys;

src/jdk.jpackage/share/classes/jdk/jpackage/internal/ApplicationLayoutUtils.java

@@ -31,16 +31,16 @@
 
 final class ApplicationLayoutUtils {
 
-    public final static ApplicationLayout PLATFORM_APPLICATION_LAYOUT;
+    public static final ApplicationLayout PLATFORM_APPLICATION_LAYOUT;
 
-    private final static ApplicationLayout WIN_APPLICATION_LAYOUT = ApplicationLayout.build()
+    private static final ApplicationLayout WIN_APPLICATION_LAYOUT = ApplicationLayout.build()
             .setAll("")
             .appDirectory("app")
             .runtimeDirectory("runtime")
             .appModsDirectory(Path.of("app", "mods"))
             .create();
 
-    private final static ApplicationLayout MAC_APPLICATION_LAYOUT = ApplicationLayout.build()
+    private static final ApplicationLayout MAC_APPLICATION_LAYOUT = ApplicationLayout.build()
             .launchersDirectory("Contents/MacOS")
             .appDirectory("Contents/app")
             .runtimeDirectory("Contents/runtime/Contents/Home")
@@ -49,7 +49,7 @@ final class ApplicationLayoutUtils {
             .contentDirectory("Contents")
             .create();
 
-    private final static ApplicationLayout LINUX_APPLICATION_LAYOUT = ApplicationLayout.build()
+    private static final ApplicationLayout LINUX_APPLICATION_LAYOUT = ApplicationLayout.build()
             .launchersDirectory("bin")
             .appDirectory("lib/app")
             .runtimeDirectory("lib/runtime")

src/jdk.jpackage/share/classes/jdk/jpackage/internal/PackageBuilder.java

@@ -244,6 +244,6 @@ private static Optional<Path> defaultInstallDir(StandardPackageType pkgType, Str
     private Path predefinedAppImage;
     private Path installDir;
 
-    final private PackageType type;
-    final private Application app;
+    private final PackageType type;
+    private final Application app;
 }

src/jdk.jpackage/share/classes/jdk/jpackage/internal/cli/StandardValueConverter.java

@@ -32,7 +32,7 @@
 
 
 final class StandardValueConverter {
-    final static ValueConverter<String> IDENTITY_CONV = new ValueConverter<>() {
+    static final ValueConverter<String> IDENTITY_CONV = new ValueConverter<>() {
         @Override
         public String convert(String value) {
             return Objects.requireNonNull(value);
@@ -44,7 +44,7 @@ public Class<? extends String> valueType() {
         }
     };
 
-    final static ValueConverter<Path> PATH_CONV = new ValueConverter<>() {
+    static final ValueConverter<Path> PATH_CONV = new ValueConverter<>() {
         @Override
         public Path convert(String value) {
             return Path.of(value);
@@ -56,7 +56,7 @@ public Class<? extends Path> valueType() {
         }
     };
 
-    final static ValueConverter<String[]> STRING_ARRAY_CONV = new ValueConverter<>() {
+    static final ValueConverter<String[]> STRING_ARRAY_CONV = new ValueConverter<>() {
         @Override
         public String[] convert(String value) {
             return value.split("[,\\s]");
@@ -68,7 +68,7 @@ public Class<? extends String[]> valueType() {
         }
     };
 
-    final static ValueConverter<Path[]> PATH_ARRAY_CONV = new ValueConverter<>() {
+    static final ValueConverter<Path[]> PATH_ARRAY_CONV = new ValueConverter<>() {
         @Override
         public Path[] convert(String value) {
             return Stream.of(value.split(File.pathSeparator)).map(Path::of).toArray(Path[]::new);