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

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


The following commit(s) were added to refs/heads/master by this push:
     new 8dd0c053 JEXL-428 : Operators.java nits;
8dd0c053 is described below

commit 8dd0c053dbcfd9ebc895e066e5938cfa6b71b6aa
Author: Henrib <hbies...@gmail.com>
AuthorDate: Sat Sep 28 16:44:31 2024 +0200

    JEXL-428 : Operators.java nits;
---
 src/main/java/org/apache/commons/jexl3/internal/Operators.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/commons/jexl3/internal/Operators.java 
b/src/main/java/org/apache/commons/jexl3/internal/Operators.java
index 693c4079..84a3077c 100644
--- a/src/main/java/org/apache/commons/jexl3/internal/Operators.java
+++ b/src/main/java/org/apache/commons/jexl3/internal/Operators.java
@@ -37,11 +37,11 @@ import org.apache.commons.jexl3.parser.JexlNode;
  */
 public final class Operators implements JexlArithmetic.Uberspect {
     /** The uberspect. */
-    protected final Uberspect uberspect;
+    private final Uberspect uberspect;
     /** The arithmetic instance being analyzed. */
-    protected final JexlArithmetic arithmetic;
+    private final JexlArithmetic arithmetic;
     /** The set of overloaded operators. */
-    protected final Set<JexlOperator> overloads;
+    private final Set<JexlOperator> overloads;
 
     /**
      * The comparison operators.
@@ -71,7 +71,7 @@ public final class Operators implements 
JexlArithmetic.Uberspect {
 
     @Override
     public JexlMethod getOperator(final JexlOperator operator, final Object... 
args) {
-        return overloads.contains(operator) && args != null
+        return overloads.contains(operator) && args != null && args.length == 
operator.getArity()
                 ? uberspectOperator(arithmetic, operator, args)
                 : null;
     }

Reply via email to