Update to Java 25, Airbase 347, ASM 9.9.1, and Modernizer 17

.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
       fail-fast: false
       matrix:
         java-version:
-          - 17
+          - 25
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-java@v4

pom.xml

@@ -5,7 +5,7 @@
     <parent>
         <groupId>io.airlift</groupId>
         <artifactId>airbase</artifactId>
-        <version>160</version>
+        <version>347</version>
     </parent>
 
     <artifactId>bytecode</artifactId>
@@ -28,10 +28,10 @@
         <air.check.skip-pmd>true</air.check.skip-pmd>
         <air.check.skip-jacoco>true</air.check.skip-jacoco>
 
-        <project.build.targetJdk>17</project.build.targetJdk>
-        <air.modernizer.java-version>8</air.modernizer.java-version>
+        <project.build.targetJdk>25</project.build.targetJdk>
+        <air.modernizer.java-version>17</air.modernizer.java-version>
 
-        <dep.asm.version>9.7</dep.asm.version>
+        <dep.asm.version>9.9.1</dep.asm.version>
     </properties>
 
     <dependencies>
@@ -76,6 +76,12 @@
             <artifactId>junit-jupiter-api</artifactId>
             <scope>test</scope>
         </dependency>
+
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-engine</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>

src/main/java/io/airlift/bytecode/ByteCodeGenerator.java

@@ -28,8 +28,8 @@
 import java.nio.file.Path;
 import java.util.Optional;
 
-import static com.google.common.io.CharStreams.nullWriter;
 import static io.airlift.bytecode.ParameterizedType.typeFromJavaClassName;
+import static java.io.Writer.nullWriter;
 import static java.nio.file.Files.createDirectories;
 import static java.util.Objects.requireNonNull;
 

src/main/java/io/airlift/bytecode/DynamicClassLoader.java

@@ -121,7 +121,7 @@ protected Class<?> loadClass(String name, boolean resolve)
 
             if (overrideClassLoader.isPresent()) {
                 try {
-                    return resolveClass(overrideClassLoader.get().loadClass(name), resolve);
+                    return resolveClass(overrideClassLoader.orElseThrow().loadClass(name), resolve);
                 }
                 catch (ClassNotFoundException e) {
                     // not in override loader

src/main/java/io/airlift/bytecode/HiddenClassGenerator.java

@@ -77,7 +77,7 @@ public <T> Class<? extends T> defineHiddenClass(ClassDefinition classDefinition,
                 definedClassLookup = lookup.defineHiddenClass(bytecode, true);
             }
             else {
-                definedClassLookup = lookup.defineHiddenClassWithClassData(bytecode, classData.get(), true);
+                definedClassLookup = lookup.defineHiddenClassWithClassData(bytecode, classData.orElseThrow(), true);
             }
         }
         catch (IllegalAccessException e) {

src/main/java/io/airlift/bytecode/Scope.java

@@ -46,7 +46,7 @@ public class Scope
     Scope(Optional<ParameterizedType> thisType, Iterable<Parameter> parameters)
     {
         if (thisType.isPresent()) {
-            thisVariable = new Variable("this", thisType.get());
+            thisVariable = new Variable("this", thisType.orElseThrow());
             variables.put("this", thisVariable);
             allVariables.add(thisVariable);
         }