You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2019/07/22 13:17:29 UTC

[GitHub] [tinkerpop] FlorianHockmann commented on issue #1163: TINKERPOP-1921 Added hasNext() to javascript and .NET

FlorianHockmann commented on issue #1163: TINKERPOP-1921 Added hasNext() to javascript and .NET
URL: https://github.com/apache/tinkerpop/pull/1163#issuecomment-513786967
 
 
   I don't have a strong opinion on this. Wrapping `IEnumerator<Traverser>` in a dedicated class that already implements `HasNext()` sounds like a good idea as it would be a better place for this logic, but we can also keep it as is in this PR if it requires too much effort to make this change.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services