From 0f16cc12da911c8d205936811a2b56ab8c19efa2 Mon Sep 17 00:00:00 2001 From: Gary Gregory Date: Sat, 25 Nov 2023 12:06:13 -0500 Subject: [PATCH] Internal refactoring --- .../org/apache/commons/logging/LogSource.java | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/main/java/org/apache/commons/logging/LogSource.java b/src/main/java/org/apache/commons/logging/LogSource.java index b58f2ff..00ac1c4 100644 --- a/src/main/java/org/apache/commons/logging/LogSource.java +++ b/src/main/java/org/apache/commons/logging/LogSource.java @@ -75,18 +75,10 @@ public class LogSource { static { // Is Log4J Available? - try { - log4jIsAvailable = null != Class.forName("org.apache.log4j.Logger"); - } catch (final Throwable t) { - log4jIsAvailable = false; - } + log4jIsAvailable = isClassForName("org.apache.log4j.Logger"); // Is JDK 1.4 Logging Available? - try { - jdk14IsAvailable = null != Class.forName("org.apache.commons.logging.impl.Jdk14Logger"); - } catch (final Throwable t) { - jdk14IsAvailable = false; - } + jdk14IsAvailable = isClassForName("org.apache.commons.logging.impl.Jdk14Logger"); // Set the default Log implementation String name = null; @@ -159,6 +151,14 @@ public class LogSource { return logs.keySet().toArray(EMPTY_STRING_ARRAY); } + private static boolean isClassForName(final String className) { + try { + return Class.forName(className) != null; + } catch (Throwable e) { + return false; + } + } + /** * Create a new {@link Log} implementation, based on the given name. *