Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/876#discussion_r202453831
--- Diff:
gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatStepTest.java
---
@@ -39,7 +39,9 @@
__.repeat(out()).times(3),
__.repeat(out().as("x")).times(3),
__.out().emit().repeat(out()).times(3),
- __.repeat(out()).until(hasLabel("x"))
+ __.repeat(out()).until(hasLabel("x")),
+ __.repeat("a", __.out()).times(3),
--- End diff --
Ah, alright, I totally missed the first entry, all good then.
We would have to shift or rotate the `loopName`'s hashCode value by the
depth of the repeat traversal. Unfortunately, we don't have this information
and I don't see how your `hashCode` implementation above would solve the
problem. Anyway, it's an unlikely corner case and I don't think we need to care
about it too much, so no big deal if we don't have a solution for now.
---
