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-jxpath.git
commit 459a318abd9a0915b3bb2e61c991d36112a95093 Author: Gary D. Gregory <[email protected]> AuthorDate: Sun Mar 16 13:29:47 2025 -0400 No need to nest else --- .../java/org/apache/commons/jxpath/ri/model/VariablePointer.java | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java index c9a97ef..964f8af 100644 --- a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java +++ b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java @@ -251,11 +251,7 @@ public class VariablePointer extends NodePointer { public NodePointer getImmediateValuePointer() { if (valuePointer == null) { Object value; - if (actual) { - value = getImmediateNode(); - valuePointer = newChildNodePointer(this, null, value); - } - else { + if (!actual) { return new NullPointer(this, getName()) { private static final long serialVersionUID = 1L; @@ -266,6 +262,8 @@ public class VariablePointer extends NodePointer { } }; } + value = getImmediateNode(); + valuePointer = newChildNodePointer(this, null, value); } return valuePointer; }
