test & cleanup

Author: crogoz

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

@@ -2240,7 +2240,6 @@ private List<Op> visitModifiers(
         builder.open(ZERO);
         boolean first = true;
         boolean lastWasAnnotation = false;
-        boolean hasExplicitParameterizedAnnotation = false;
         while (!annotations.isEmpty()) {
             if (nextIsModifier()) {
                 break;
@@ -2257,7 +2256,7 @@ private List<Op> visitModifiers(
         }
         builder.close();
 
-        // pjf specific: enforce breaking operatins for record parameters with annotations with explicit parameters
+        // pjf specific: record params should take into consideration the columnLimit
         ImmutableList<Op> trailingBreak = isRecordParameter
                 ? ImmutableList.of(Break.builder()
                         .fillMode(FillMode.UNIFIED)

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/Records.input

@@ -26,4 +26,10 @@ class Records {
       this.x = x;
     }
   }
+
+  record R10(int a, int b, int c, int d, int e, int f, int g, int h, int i, int j, int k, int l, int m, int n, int p) {
+  }
+
+  record R10(int a, int b, int c, int d, int e, int f, int g, int h, int i, int j, int k, int l) { }
+
 }

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

@@ -26,4 +26,9 @@ class Records {
             this.x = x;
         }
     }
+
+    record R10(
+            int a, int b, int c, int d, int e, int f, int g, int h, int i, int j, int k, int l, int m, int n, int p) {}
+
+    record R10(int a, int b, int c, int d, int e, int f, int g, int h, int i, int j, int k, int l) {}
 }