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

Reply via email to