From 858d8a90ff34b6884e7cebde64af3ae6e8b74359 Mon Sep 17 00:00:00 2001 From: Louis Bergelson Date: Mon, 18 Dec 2023 16:08:25 -0500 Subject: [PATCH] Merging duplicate implmementations of isConcrete (#197) --- .../broadinstitute/barclay/argparser/ClassFinder.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/broadinstitute/barclay/argparser/ClassFinder.java b/src/main/java/org/broadinstitute/barclay/argparser/ClassFinder.java index 024d9cf..e0ebd3c 100644 --- 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> 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); } } \ No newline at end of file