lb update to 21 (#198)

* Updating to java 21

* replacing use of deprecated function
* Suppressing some 'this-escape' warnings
* update github actions to use java 21

.github/workflows/tests.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        java: [ 17 ]
+        java: [ 21 ]
         experimental: [false]
       fail-fast: false
     continue-on-error: ${{ matrix.experimental }}

build.gradle

@@ -11,8 +11,9 @@ wrapper {
 }
 
 java {
-    sourceCompatibility = JavaVersion.VERSION_17
-    targetCompatibility = JavaVersion.VERSION_17
+    toolchain {
+        languageVersion = JavaLanguageVersion.of(21)
+    }
 }
 
 group = 'org.broadinstitute'

src/main/java/org/broadinstitute/barclay/argparser/ArgumentDefinition.java

@@ -29,6 +29,7 @@ public abstract class ArgumentDefinition {
      * @param containingObject the parent {@code Object} containing the {@code Field} for this argument. cannot be null.
      * @param underlyingField the {@code Field} object for this argument. cannot be null.
      */
+    @SuppressWarnings("this-escape")
     public ArgumentDefinition(final Object containingObject, final Field underlyingField) {
         Utils.nonNull(underlyingField, "An underlying field must be provided");
         Utils.nonNull(containingObject, "A containing object must be provided");

src/main/java/org/broadinstitute/barclay/argparser/ClassFinder.java

@@ -10,6 +10,7 @@
 import java.net.URLClassLoader;
 import java.net.URLDecoder;
 import java.nio.charset.StandardCharsets;
+import java.nio.file.Paths;
 import java.util.Enumeration;
 import java.util.LinkedHashSet;
 import java.util.Set;
@@ -44,7 +45,7 @@ public ClassFinder(final File jarFile) throws IOException {
         // but the jarPath is remembered so that the iteration over the classpath skips anything other than
         // the jarPath.
         jarPath = jarFile.getCanonicalPath();
-        final URL[] urls = {new URL("file", "", jarPath)};
+        final URL[] urls = {Paths.get(jarPath).toUri().toURL()};
         loader = new URLClassLoader(urls, Thread.currentThread().getContextClassLoader());
     }
 

src/main/java/org/broadinstitute/barclay/argparser/NamedArgumentDefinition.java

@@ -42,6 +42,7 @@ public class NamedArgumentDefinition extends ArgumentDefinition {
      *                                      {@code CommandLinePluginDescriptor} itself are not contained in a plugin,
      *                                      and will not have a {@code descriptorForControllingPlugin}. can be null.
      */
+    @SuppressWarnings("this-escape")
     public NamedArgumentDefinition(
             final Argument argumentAnnotation,
             final Object containingObject,

src/main/java/org/broadinstitute/barclay/argparser/PositionalArgumentDefinition.java

@@ -22,6 +22,7 @@ public class PositionalArgumentDefinition extends ArgumentDefinition {
      * @param containingObject the parent {@code Object} containing this argument. cannot be null.
      * @param argField the {@code Field} for this argument. cannot be null.
      */
+    @SuppressWarnings("this-escape")
     public PositionalArgumentDefinition(
             final PositionalArguments positionalArgumentsAnnotation,
             final Object containingObject,

src/main/java/org/broadinstitute/barclay/help/DocWorkUnit.java

@@ -48,6 +48,7 @@ public class DocWorkUnit implements Comparable<DocWorkUnit> {
      * @param clazz
      * @param documentedFeatureAnnotation
      */
+    @SuppressWarnings("this-escape")
     public DocWorkUnit(
             final DocWorkUnitHandler workUnitHandler,
             final Element docElement,