Merging duplicate implmementations of isConcrete (#197)

broadinstitute · Dec 18, 2023 · 858d8a9 · 858d8a9
1 parent 84256bd
commit 858d8a9
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/main/java/org/broadinstitute/barclay/argparser/ClassFinder.java b/src/main/java/org/broadinstitute/barclay/argparser/ClassFinder.java
@@ -1,16 +1,18 @@
 package org.broadinstitute.barclay.argparser;
 
-import org.apache.logging.log4j.Logger;
 import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
+import org.broadinstitute.barclay.utils.JVMUtils;
 
 import java.io.File;
 import java.io.IOException;
-import java.lang.reflect.Modifier;
 import java.net.URL;
 import java.net.URLClassLoader;
 import java.net.URLDecoder;
 import java.nio.charset.StandardCharsets;
-import java.util.*;
+import java.util.Enumeration;
+import java.util.LinkedHashSet;
+import java.util.Set;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipFile;
 
@@ -185,6 +187,6 @@ public Set<Class<?>> getConcreteClasses() {
      * @return true if the class is neither abstract nor an interface, otherwise false
      */
     public static boolean isConcrete( final Class<?> clazz ) {
-        return ! Modifier.isAbstract(clazz.getModifiers()) && ! Modifier.isInterface(clazz.getModifiers());
+       return JVMUtils.isConcrete(clazz);
     }
 }