cleanup

Author: crogoz

idea-plugin/build.gradle

@@ -22,7 +22,7 @@ intellij {
     pluginName = name
     updateSinceUntilBuild = true
     version = "2024.1"
-    plugins = [ 'java' ]
+    plugins = ['java']
 }
 
 patchPluginXml {

palantir-java-format/src/main/java/com/palantir/javaformat/java/JavaInputAstVisitor.java

@@ -2409,8 +2409,7 @@ protected void visitFormals(Optional<VariableTree> receiver, List<? extends Vari
         }
         boolean isRecordParams = false;
         if (!parameters.isEmpty() && parameters.get(0) instanceof JCTree.JCVariableDecl) {
-            JCTree.JCVariableDecl param = (JCTree.JCVariableDecl) parameters.get(0);
-            isRecordParams = (param.mods.flags & RECORD) == RECORD;
+            isRecordParams = isInRecord(((JCTree.JCVariableDecl) parameters.get(0)).mods);
         }
 
         builder.open(ZERO);
@@ -3499,7 +3498,13 @@ int declareOne(
                 new ArrayDeque<>(typeWithDims.isPresent() ? typeWithDims.get().dims : Collections.emptyList());
         int baseDims = 0;
 
-        builder.open(ZERO);
+        builder.open(
+                kind == DeclarationKind.PARAMETER
+                                && (modifiers.isPresent()
+                                        && !modifiers.get().getAnnotations().isEmpty()
+                                        && !isInRecord(modifiers.get()))
+                        ? plusFour
+                        : ZERO);
         {
             if (modifiers.isPresent()) {
                 visitAndBreakModifiers(modifiers.get(), annotationsDirection, Optional.of(verticalAnnotationBreak));
@@ -3869,6 +3874,13 @@ private Direction inlineAnnotationDirection(ModifiersTree modifiers) {
         return Direction.HORIZONTAL;
     }
 
+    /**
+     * Checks if the modifiers are parameters of a record definition.
+     */
+    private boolean isInRecord(ModifiersTree modifiers) {
+        return (((JCTree.JCModifiers) modifiers).flags & RECORD) == RECORD;
+    }
+
     /**
      * Emit a {@link Token}.
      *

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/B23804934.output

@@ -6,7 +6,7 @@ class B23804934 {
             ZzzzzzzZzzzzzzz<ZzzzzzZzzzZzzz> zzzzzzZzzzZzzzZzzzzzzz,
             @ZzzzZzzzzZzzzzzz Zzzzzz zzzzzZzzzzzz,
             @ZzzzzzzzzzzZzzzzzz(Zz.ZZZZZZZ_ZZZZZ_ZZZZ_ZZZZZZ_ZZZZ_ZZZZZZZZZZ)
-            Zzzzzzzz<Zzzzzzz> zzzzzZzzzZzzzzzZzzzZzzzzzzzzz) {
+                    Zzzzzzzz<Zzzzzzz> zzzzzZzzzZzzzzzZzzzZzzzzzzzzz) {
         zzzz.zzzzzzzzZzzzzzZzzzzZzzzzzzz = zzzzzzzzZzzzzzZzzzzZzzzzzzz;
         zzzz.zzzzzzZzzzZzzzZzzzzzzz = zzzzzzZzzzZzzzZzzzzzzz;
         zzzz.zzzzzZzzzzzz = zzzzzZzzzzzz;
@@ -16,7 +16,7 @@ class B23804934 {
     zzzz z(
             zzz z,
             @Zzzzzzzz @Zzzzzzzzzz
-            ZzzzzzzzzZzzz<Zzzzzz> zzzzZzzzZzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
+                    ZzzzzzzzzZzzz<Zzzzzz> zzzzZzzzZzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
 
     @ZzzZzzzZzzzzzzzzzz
     @Zzzzzzzz(zzzzzzzzz = Zzzzzzzzz.ZZ)

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/B24702438.output

@@ -48,16 +48,16 @@ class B24702438 {
     void f(
             int a,
             @Nullable @Deprecated
-            ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
+                    ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
             @Nullable @Deprecated
-            ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
+                    ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
             @Nullable @Deprecated
-            ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
+                    ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
             int c) {}
 
     void g(
             @Nullable @Deprecated
-            ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
+                    ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
             @Nullable @Deprecated
-            ImmutableList<String> veryVeryLoooooooooooooooooooooooooooooooooooooooooooooooooooooooong) {}
+                    ImmutableList<String> veryVeryLoooooooooooooooooooooooooooooooooooooooooooooooooooooooong) {}
 }

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/TryWithResources.output

@@ -12,7 +12,7 @@ class TryWtihResources {
         }
 
         try (@SuppressWarnings("resource")
-        Scanner inputScanner = new Scanner(inputStream).useDelimiter("\\s+|,")) {
+                Scanner inputScanner = new Scanner(inputStream).useDelimiter("\\s+|,")) {
             while (inputScanner.hasNextLong()) {
                 placementIds.add(inputScanner.nextLong());
             }