-
Notifications
You must be signed in to change notification settings - Fork 504
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: the vertex from edges is missing properties #604
Conversation
fix: #595 Change-Id: I7cacb37f7d7592cedbdcc6f98db7cc8bbb1bf537
boolean exist = edges.hasNext(); | ||
if (!exist && !throwIfNotExist) { | ||
return false; | ||
} | ||
E.checkState(edges.hasNext(), "Edge '%s' does not exist", this.id); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
edges.hasNext() -> exits
Change-Id: I507c5ded936374a5df18858a89f41b733103f1ed
Codecov Report
@@ Coverage Diff @@
## master #604 +/- ##
============================================
+ Coverage 70.46% 70.73% +0.26%
- Complexity 3458 3480 +22
============================================
Files 216 216
Lines 16612 16618 +6
Branches 2367 2369 +2
============================================
+ Hits 11705 11754 +49
+ Misses 3658 3618 -40
+ Partials 1249 1246 -3
Continue to review full report at Codecov.
|
75f3d49
to
b12b54b
Compare
b12b54b
to
878b739
Compare
fix: #595
Change-Id: I7cacb37f7d7592cedbdcc6f98db7cc8bbb1bf537