[openrewrite] initial merge and Java8toJava11

Signed-off-by: Vincent Potucek <vpotucek@me.com>

Author: Vincent Potucek

.github/workflows/ci.yml

@@ -90,6 +90,9 @@ jobs:
       - name: 'Test'
         shell: bash
         run: mvn test -B
+      - name: 'SanityCheck'
+        shell: bash
+        run: mvn rewrite:dryRun
       - name: 'Javadoc'
         shell: bash
         run: mvn -P '!examples' javadoc:javadoc

.gitignore

@@ -8,8 +8,9 @@
 
 # intellij
 .idea/ant.xml
-.idea/codestream.xml
+.idea/checkstyle-idea.xml
 .idea/codeStyleSettings.xml
+.idea/codestream.xml
 .idea/compiler.xml
 .idea/copyright
 .idea/dataSources.ids
@@ -21,6 +22,7 @@
 .idea/libraries
 .idea/misc.xml
 .idea/modules.xml
+.idea/palantir-java-format.xml
 .idea/shelf
 .idea/tasks.xml
 .idea/uiDesigner.xml

annotation/pom.xml

@@ -24,8 +24,8 @@
     <version>1.0-HEAD-SNAPSHOT</version>
   </parent>
 
-  <name>@BugPattern annotation</name>
   <artifactId>error_prone_annotation</artifactId>
+  <name>@BugPattern annotation</name>
 
   <licenses>
     <license>

annotations/pom.xml

@@ -24,8 +24,15 @@
     <version>1.0-HEAD-SNAPSHOT</version>
   </parent>
 
-  <name>error-prone annotations</name>
   <artifactId>error_prone_annotations</artifactId>
+  <name>error-prone annotations</name>
+
+  <licenses>
+    <license>
+      <name>Apache 2.0</name>
+      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
+    </license>
+  </licenses>
 
   <dependencies>
     <dependency>
@@ -36,18 +43,12 @@
     </dependency>
   </dependencies>
 
-  <licenses>
-    <license>
-      <name>Apache 2.0</name>
-      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
-    </license>
-  </licenses>
-
   <build>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
+        <version>3.14.1</version>
         <configuration>
           <compilerArgs combine.self="override" />
         </configuration>
@@ -79,6 +80,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
+        <version>3.4.2</version>
         <configuration>
           <archive>
             <manifestEntries>

check_api/pom.xml

@@ -24,8 +24,8 @@
     <version>1.0-HEAD-SNAPSHOT</version>
   </parent>
 
-  <name>error-prone check api</name>
   <artifactId>error_prone_check_api</artifactId>
+  <name>error-prone check api</name>
 
   <licenses>
     <license>
@@ -49,6 +49,12 @@
       <artifactId>jspecify</artifactId>
       <version>${jspecify.version}</version>
     </dependency>
+    <dependency>
+      <groupId>jakarta.annotation</groupId>
+      <artifactId>jakarta.annotation-api</artifactId>
+      <version>1.3.5</version>
+      <scope>provided</scope>
+    </dependency>
     <dependency>
       <!-- GPLv2 with Classpath Exception -->
       <groupId>io.github.eisop</groupId>
@@ -103,6 +109,12 @@
       <version>${truth.version}</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>jakarta.inject</groupId>
+      <artifactId>jakarta.inject-api</artifactId>
+      <version>1.0.3</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <!-- MIT -->
       <groupId>org.mockito</groupId>
@@ -149,6 +161,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
+        <version>3.14.1</version>
         <configuration>
           <annotationProcessorPaths>
             <path>
@@ -189,7 +202,7 @@
                 <version>24</version>
               </jdkToolchain>
               <compileSourceRoots>
-                <compileSourceRoot>${basedir}/src/main/java24</compileSourceRoot>
+                <compileSourceRoot>${project.basedir}/src/main/java24</compileSourceRoot>
               </compileSourceRoots>
               <!-- multiReleaseOutput requires setting release -->
               <outputDirectory>${project.build.outputDirectory}/META-INF/versions/24</outputDirectory>

check_api/src/main/java/com/google/errorprone/apply/DescriptionBasedDiff.java

@@ -26,7 +26,7 @@
 import com.sun.tools.javac.tree.EndPosTable;
 import com.sun.tools.javac.tree.JCTree.JCCompilationUnit;
 import java.net.URI;
-import java.nio.file.Paths;
+import java.nio.file.Path;
 import java.util.LinkedHashSet;
 import java.util.Objects;
 import java.util.Set;
@@ -68,7 +68,7 @@ private DescriptionBasedDiff(
     URI sourceFileUri = compilationUnit.getSourceFile().toUri();
     this.sourcePath =
         (sourceFileUri.isAbsolute() && Objects.equals(sourceFileUri.getScheme(), "file"))
-            ? Paths.get(sourceFileUri).toAbsolutePath().toString()
+            ? Path.of(sourceFileUri).toAbsolutePath().toString()
             : sourceFileUri.getPath();
     this.ignoreOverlappingFixes = ignoreOverlappingFixes;
     this.importsToAdd = new LinkedHashSet<>();

check_api/src/main/java/com/google/errorprone/dataflow/nullnesspropagation/inference/InferredNullability.java

@@ -87,20 +87,18 @@ private Optional<Nullness> getNullness(InferenceVariable iv) {
       result =
           constraintGraph.predecessors(iv).stream()
               .map(this::getNullness)
-              .filter(Optional::isPresent)
-              .map(Optional::get)
+              .flatMap(Optional::stream)
               .reduce(Nullness::leastUpperBound); // use least upper bound (lub) to combine
       // 2. If not, resolve successors and use them as upper bounds
-      if (!result.isPresent()) {
+      if (result.isEmpty()) {
         result =
             constraintGraph.successors(iv).stream()
                 .map(this::getNullness)
-                .filter(Optional::isPresent)
-                .map(Optional::get)
+                .flatMap(Optional::stream)
                 .reduce(Nullness::greatestLowerBound); // use greatest lower bound (glb) to combine
       }
 
-      checkState(!inferredMemoTable.put(iv, result).isPresent());
+      checkState(inferredMemoTable.put(iv, result).isEmpty());
       return result;
     }
   }

check_api/src/main/java/com/google/errorprone/dataflow/nullnesspropagation/inference/NullnessQualifierInference.java

@@ -153,14 +153,14 @@ private void generateConstraintsFromAnnotations(
 
     Optional<Nullness> fromAnnotations =
         extractExplicitNullness(declaredType, argSelector.isEmpty() ? decl : null);
-    if (!fromAnnotations.isPresent()) {
+    if (fromAnnotations.isEmpty()) {
       // Check declared type before inferred type so that type annotations on the declaration take
       // precedence (just like declaration annotations) over annotations on the inferred type.
       // For instance, we want a @Nullable T m() to take precedence over annotations on T's inferred
       // type (e.g., @NotNull String), whether @Nullable is a declaration or type annotation.
       fromAnnotations = NullnessAnnotations.fromAnnotationsOn(inferredType);
     }
-    if (!fromAnnotations.isPresent()) {
+    if (fromAnnotations.isEmpty()) {
       if (declaredType instanceof TypeVariable typeVariable) {
         // Check bounds second so explicit annotations take precedence. Even for bounds we still use
         // equality constraint below since we have to assume the bound as the "worst" case.
@@ -482,7 +482,7 @@ private void generateConstraintsForWrite(
     boolean isBound = false;
     Optional<Nullness> fromAnnotations =
         extractExplicitNullness(lType, argSelector.isEmpty() ? decl : null);
-    if (!fromAnnotations.isPresent()) {
+    if (fromAnnotations.isEmpty()) {
       if (lType instanceof TypeVariable typeVariable) {
         fromAnnotations = NullnessAnnotations.getUpperBound(typeVariable);
         isBound = true;

check_api/src/main/java/com/google/errorprone/fixes/SuggestedFixes.java

@@ -128,7 +128,6 @@
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.stream.IntStream;
 import java.util.stream.Stream;
-import java.util.stream.StreamSupport;
 import javax.lang.model.element.Element;
 import javax.lang.model.element.ElementKind;
 import javax.lang.model.element.Modifier;
@@ -1097,7 +1096,7 @@ private static List<? extends AnnotationTree> findAnnotationsTree(Tree tree) {
   }
 
   private static @Nullable Tree suppressibleNode(TreePath path, VisitorState state) {
-    return StreamSupport.stream(path.spliterator(), false)
+    return Streams.stream(path)
         .filter(
             tree ->
                 tree instanceof MethodTree
@@ -1132,7 +1131,7 @@ public static SuggestedFix.Builder addValuesToAnnotationArgument(
                   .replaceFirst("\\(\\)", "(" + parameterPrefix + newArgument(newValues) + ")"));
     }
     Optional<ExpressionTree> maybeExistingArgument = findArgument(annotation, parameterName);
-    if (!maybeExistingArgument.isPresent()) {
+    if (maybeExistingArgument.isEmpty()) {
       return SuggestedFix.builder()
           .prefixWith(
               annotation.getArguments().getFirst(),
@@ -1192,7 +1191,7 @@ public static SuggestedFix.Builder updateAnnotationArgumentValues(
                   + ')');
     }
     Optional<ExpressionTree> maybeExistingArgument = findArgument(annotation, parameterName);
-    if (!maybeExistingArgument.isPresent()) {
+    if (maybeExistingArgument.isEmpty()) {
       return SuggestedFix.builder()
           .prefixWith(
               annotation.getArguments().getFirst(),

check_api/src/main/java/com/google/errorprone/matchers/method/MethodInvocationMatcher.java

@@ -483,7 +483,7 @@ public static Matcher<ExpressionTree> compile(Iterable<Rule> rules) {
       // First collect all the nodes that accept any token at all
       for (Node node : curr) {
         for (Map.Entry<Optional<Token>, Node> entry : nfa.row(node).entrySet()) {
-          if (!entry.getKey().isPresent()) {
+          if (entry.getKey().isEmpty()) {
             acceptsAny.add(entry.getValue());
           }
         }