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-jexl.git

commit 8e9e651a91a579988552aae7f550feb483465c23
Author: Gary Gregory <[email protected]>
AuthorDate: Sat Jun 1 10:29:27 2024 -0400

    Use else if
---
 src/main/java/org/apache/commons/jexl3/internal/Engine.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/commons/jexl3/internal/Engine.java 
b/src/main/java/org/apache/commons/jexl3/internal/Engine.java
index 4dc8658e..95f5f450 100644
--- a/src/main/java/org/apache/commons/jexl3/internal/Engine.java
+++ b/src/main/java/org/apache/commons/jexl3/internal/Engine.java
@@ -890,10 +890,8 @@ public class Engine extends JexlEngine {
                     ns.put(namespaceName, clazz);
                 }
             }
-        } else {
-            if (logger.isWarnEnabled()) {
-                logger.warn(key + ": ambiguous declaration " + value);
-            }
+        } else if (logger.isWarnEnabled()) {
+            logger.warn(key + ": ambiguous declaration " + value);
         }
     }
 

Reply via email to