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; }