Java 21: review reformart code + default gradle jdk compatibility

Author: npeters

build.gradle

@@ -50,7 +50,8 @@ subprojects {
     apply plugin: 'java-library'
     apply plugin: 'org.inferred.processors'
 
-
+    sourceCompatibility = 11
+    targetCompatibility = 11
 
     tasks.withType(Checkstyle).configureEach {
         enabled = false

gradle-palantir-java-format/build.gradle

@@ -10,10 +10,7 @@ configurations {
         canBeResolved = true
     }
 }
-java{
-    sourceCompatibility = 11
-    targetCompatibility = 11
-}
+
 
 dependencies {
     compileOnly 'com.diffplug.spotless:spotless-plugin-gradle'

idea-plugin/build.gradle

@@ -61,10 +61,6 @@ configurations {
     }
 }
 
-java{
-    sourceCompatibility = 11
-    targetCompatibility = 11
-}
 
 dependencies {
     implementation project(':palantir-java-format-jdk-bootstrap')

palantir-java-format-jdk-bootstrap/build.gradle

@@ -2,11 +2,6 @@ apply plugin: 'java-library'
 apply plugin: 'com.palantir.external-publish-jar'
 apply plugin: 'com.palantir.revapi'
 
-java{
-    sourceCompatibility = 11
-    targetCompatibility = 11
-}
-
 dependencies {
     annotationProcessor "org.immutables:value"
 

palantir-java-format-spi/build.gradle

@@ -2,10 +2,6 @@ apply plugin: 'java-library'
 apply plugin: 'com.palantir.external-publish-jar'
 apply plugin: 'com.palantir.revapi'
 
-java{
-    sourceCompatibility = 11
-    targetCompatibility = 11
-}
 
 dependencies {
     api 'com.google.guava:guava'

palantir-java-format/build.gradle

@@ -53,6 +53,7 @@ tasks.withType(JavaCompile).configureEach {
     if (JavaVersion.current() < JavaVersion.VERSION_14) {
         excludes = ['**/Java14InputAstVisitor.java']
     }
+
 }
 
 tasks.withType(Test).configureEach {
@@ -94,8 +95,7 @@ tasks.named("jar", Jar) {
 
 tasks.named('jar', Jar){
     // hack to get java21 Class ...
-    // the build script should be completely review to handle java 21
+    // the build script should be completely rewritten to handle java 21
     from ('build/classes/java/main', '../palantir-java-format21/build/classes/java/main')
-
     include('**/*')
 }

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

@@ -131,7 +131,6 @@ static JavaOutput format(
         OpsBuilder opsBuilder = new OpsBuilder(javaInput);
 
         JavaInputAstVisitor visitor;
-
         if (getRuntimeVersion() >= 21) {
             try {
                 visitor = Class.forName("com.palantir.javaformat.java.java21.Java21InputAstVisitor")

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

@@ -447,7 +447,6 @@ public CharSequence getCharContent(boolean ignoreEncodingErrors) throws IOExcept
                 if (stringFragmentKind) {
                     stringFragmentEndPos = t.endPos();
                 }
-
             } else if (tokText.startsWith("//") || tokText.startsWith("/*")) {
                 // For compatibility with an earlier lexer, the newline after a // comment is its own tok.
                 if (tokText.startsWith("//") && (originalTokText.endsWith("\n") || originalTokText.endsWith("\r"))) {

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

@@ -394,7 +394,6 @@ protected void handleModule(boolean first, CompilationUnitTree node) {}
 
     /** Skips over extra semi-colons at the top-level, or in a class member declaration lists. */
     protected void dropEmptyDeclarations() {
-
         if (builder.peekToken().equals(Optional.of(";"))) {
             while (builder.peekToken().equals(Optional.of(";"))) {
                 builder.forcedBreak();

palantir-java-format21/src/main/java/com/palantir/javaformat/java/java21/Java21InputAstVisitor.java

@@ -78,8 +78,6 @@ public Void visitDeconstructionPattern(DeconstructionPatternTree node, Void unus
 
         Void r = scan(node.getDeconstructor(), unused);
         token("(");
-
-        // r = scanAndReduceVoid(, unused, r);
         boolean firstInRow = true;
         for (PatternTree item : node.getNestedPatterns()) {
             if (!firstInRow) {