(commons-logging) 03/04: Collapse into if-else

[ggregory]

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-logging.git

commit 618ab14b4a526680de2853cc40755a80af9b80e6
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Jun 30 07:49:26 2024 -0400

    Collapse into if-else
---
 src/main/java/org/apache/commons/logging/LogFactory.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/commons/logging/LogFactory.java 
b/src/main/java/org/apache/commons/logging/LogFactory.java
index ce68c2f..c8cfa7a 100644
--- a/src/main/java/org/apache/commons/logging/LogFactory.java
+++ b/src/main/java/org/apache/commons/logging/LogFactory.java
@@ -898,10 +898,8 @@ public abstract class LogFactory {
                     factory = newFactory(factoryClass, baseClassLoader, 
contextClassLoader);
 
                     // TODO: think about whether we need to handle exceptions 
from newFactory
-                } else {
-                    if (isDiagnosticsEnabled()) {
-                        logDiagnostic("[LOOKUP] Properties file has no entry 
specifying LogFactory subclass.");
-                    }
+                } else if (isDiagnosticsEnabled()) {
+                    logDiagnostic("[LOOKUP] Properties file has no entry 
specifying LogFactory subclass.");
                 }
             } else if (isDiagnosticsEnabled()) {
                 logDiagnostic("[LOOKUP] No properties file available to 
determine" + " LogFactory subclass from..");