This is an automated email from the ASF dual-hosted git repository. spmallette pushed a commit to branch TINKERPOP-2235 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit c102bc3fa82ab99aec2f305046a3640ba867c941 Author: stephen <[email protected]> AuthorDate: Mon Nov 18 19:22:14 2019 -0500 TINKERPOP-2235 Fixed ImmutablePath again Can't use null for the currentObject - has to be some kind of special indicator. Still not convinced that this is the best way to make this work, but so much of our infrastructure is predicated on null having some special meaning rather than having some property that describes its state. --- .../tinkerpop/gremlin/process/traversal/step/util/ImmutablePath.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ImmutablePath.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ImmutablePath.java index 1f38ece..1b5e562 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ImmutablePath.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ImmutablePath.java @@ -36,7 +36,8 @@ import java.util.Set; */ public class ImmutablePath implements Path, Serializable, Cloneable { - private static final ImmutablePath TAIL_PATH = new ImmutablePath(null, null, null); + private static final Object END = EmptyPath.instance(); + private static final ImmutablePath TAIL_PATH = new ImmutablePath(null, END, null); private ImmutablePath previousPath; private Object currentObject; @@ -59,7 +60,7 @@ public class ImmutablePath implements Path, Serializable, Cloneable { } private final boolean isTail() { - return null == this.currentObject; + return END.equals(this.currentObject); } @Override
