Ran ./bin/blessed-modifier-order.sh

Author: alexeysemenyukoracle

test/jdk/tools/jpackage/helpers-test/jdk/jpackage/test/PackageTestTest.java

@@ -97,12 +97,12 @@ CountingBundleVerifier createBundleVerifier(int jpackageExitCode) {
         private final int tickCount;
     }
 
-    private final static int ERROR_EXIT_CODE_JPACKAGE = 35;
-    private final static int ERROR_EXIT_CODE_INSTALL = 27;
+    private static final int ERROR_EXIT_CODE_JPACKAGE = 35;
+    private static final int ERROR_EXIT_CODE_INSTALL = 27;
 
-    private final static CallbackFactory NEVER = new CallbackFactory(0);
-    private final static CallbackFactory ONCE = new CallbackFactory(1);
-    private final static CallbackFactory TWICE = new CallbackFactory(2);
+    private static final CallbackFactory NEVER = new CallbackFactory(0);
+    private static final CallbackFactory ONCE = new CallbackFactory(1);
+    private static final CallbackFactory TWICE = new CallbackFactory(2);
 
     enum BundleVerifier {
         ONCE_SUCCESS(ONCE),
@@ -238,7 +238,7 @@ public String toString() {
         private final int expectedJPackageExitCode;
     }
 
-    private final static class CountingInstaller extends TickCounter implements Function<JPackageCommand, Integer> {
+    private static final class CountingInstaller extends TickCounter implements Function<JPackageCommand, Integer> {
 
         @Override
         public Integer apply(JPackageCommand cmd) {
@@ -439,7 +439,7 @@ private List<Verifiable> run(Optional<Consumer<PackageTest>> customConfigure) {
         }
     }
 
-    private final static class TestSpecBuilder {
+    private static final class TestSpecBuilder {
 
         TestSpecBuilder type(PackageType v) {
             type = Objects.requireNonNull(v);

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/PackageTest.java

@@ -618,7 +618,7 @@ private boolean unpackNotSupported() {
             return processed(Action.UNPACK) && packageHandlers.unpackHandler().isEmpty();
         }
 
-        private final static class State {
+        private static final class State {
             private final Set<Action> packageActions = new HashSet<>();
             private final List<Path> deleteUnpackDirs = new ArrayList<>();
         }

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/TestInstance.java

@@ -137,7 +137,7 @@ private static String formatArgs(List<Object> values) {
             private List<Object> methodArgs;
             private Method method;
 
-            private final static HexFormat ARGS_CHAR_FORMATTER = HexFormat.of().withUpperCase();
+            private static final HexFormat ARGS_CHAR_FORMATTER = HexFormat.of().withUpperCase();
         }
 
         static TestDesc create(Method m, Object... args) {

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/WindowsHelper.java

@@ -662,7 +662,7 @@ Path getPath() {
         private final RegValuePath reg;
         private final Optional<SpecialFolderDotNet> alt;
 
-        private final static Map<SpecialFolder, Path> CACHE = new ConcurrentHashMap<>();
+        private static final Map<SpecialFolder, Path> CACHE = new ConcurrentHashMap<>();
     }
 
     private static final class ShortPathUtils {

test/jdk/tools/jpackage/junit/share/jdk.jpackage/jdk/jpackage/internal/util/TokenReplaceTest.java

@@ -181,7 +181,7 @@ public static Stream<TestSpec> test() {
         ).map(TestSpec.Builder::create);
     }
 
-    private final static class CountingSupplier implements Supplier<Object> {
+    private static final class CountingSupplier implements Supplier<Object> {
 
         CountingSupplier(Object value, int expectedCount) {
             this.value = value;

test/jdk/tools/jpackage/linux/LinuxResourceTest.java

@@ -130,7 +130,7 @@ public static void testHardcodedProperties() throws IOException {
         .run(Action.CREATE);
     }
 
-    private final static class PropertyValidator {
+    private static final class PropertyValidator {
 
         PropertyValidator name(String v) {
             name = v;

test/jdk/tools/jpackage/share/ErrorTest.java

@@ -289,7 +289,7 @@ public final String toString() {
             return sb.toString();
         }
 
-        private final static String DEFAULT_APP_DESC = "Hello";
+        private static final String DEFAULT_APP_DESC = "Hello";
     }
 
     private static TestSpec.Builder testSpec() {

test/jdk/tools/jpackage/share/JavaOptionsEqualsTest.java

@@ -54,13 +54,13 @@
 
 public class JavaOptionsEqualsTest {
 
-    private final static String OPTION1 =
+    private static final String OPTION1 =
         "--add-exports=java.base/sun.util=me.mymodule.foo,ALL-UNNAMED";
-    private final static String OPTION2 =
+    private static final String OPTION2 =
         "--add-exports=java.base/sun.security.util=other.mod.bar,ALL-UNNAMED";
-    private final static String WARNING1 =
+    private static final String WARNING1 =
         "WARNING: Unknown module: me.mymodule.foo";
-    private final static String WARNING2 =
+    private static final String WARNING2 =
         "WARNING: Unknown module: other.mod.bar";
 
     private final JPackageCommand cmd;

test/jdk/tools/jpackage/share/ModulePathTest.java

@@ -138,7 +138,7 @@ public void test(String javaAppDesc) throws IOException {
 
     private final List<String> modulePathArgs;
 
-    private final static String GOOD_PATH = "@GoodPath@";
-    private final static String EMPTY_DIR = "@EmptyDir@";
-    private final static String NON_EXISTING_DIR = "@NonExistingDir@";
+    private static final String GOOD_PATH = "@GoodPath@";
+    private static final String EMPTY_DIR = "@EmptyDir@";
+    private static final String NON_EXISTING_DIR = "@NonExistingDir@";
 }