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 6ad8f314007e7c0f30aaee3422958c63709574d8 Author: Gary Gregory <[email protected]> AuthorDate: Tue May 21 13:32:31 2024 -0400 Fix PM AvoidReassigningParameters --- src/main/java/org/apache/commons/jexl3/internal/Engine32.java | 2 +- src/main/java/org/apache/commons/jexl3/internal/InterpreterBase.java | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/apache/commons/jexl3/internal/Engine32.java b/src/main/java/org/apache/commons/jexl3/internal/Engine32.java index 14199c38..89d22c8a 100644 --- a/src/main/java/org/apache/commons/jexl3/internal/Engine32.java +++ b/src/main/java/org/apache/commons/jexl3/internal/Engine32.java @@ -43,7 +43,7 @@ public class Engine32 extends Engine { /** * Static delegation of isTernaryProtected. * @param ii the interpreter (unused) - * @param node the node + * @param startNode the node * @return true if node is navigation-safe, false otherwise */ static boolean isTernaryProtected(final Interpreter ii, final JexlNode startNode) { diff --git a/src/main/java/org/apache/commons/jexl3/internal/InterpreterBase.java b/src/main/java/org/apache/commons/jexl3/internal/InterpreterBase.java index 30330426..e9e9c4ba 100644 --- a/src/main/java/org/apache/commons/jexl3/internal/InterpreterBase.java +++ b/src/main/java/org/apache/commons/jexl3/internal/InterpreterBase.java @@ -506,7 +506,8 @@ public abstract class InterpreterBase extends ParserVisitor { * </p> * @return true if nullable variable, false otherwise */ - protected boolean isTernaryProtected(JexlNode node) { + protected boolean isTernaryProtected(final JexlNode startNode) { + JexlNode node = startNode; for (JexlNode walk = node.jjtGetParent(); walk != null; walk = walk.jjtGetParent()) { // protect only the condition part of the ternary if (walk instanceof ASTTernaryNode
