-
Notifications
You must be signed in to change notification settings - Fork 215
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
[distance] Add support for DG/GC in distance(). #922
Conversation
}, it->second); | ||
|
||
// The smallest possible distance found, end searching. | ||
if (! is_first && result <= zero) |
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.
is it possible that result
is negative?
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.
Technically it is, a (potentially user-defined) strategy can return any value. Possible reason may be an error or floating-point issue.
box2_t b2 = geometry::return_envelope<box2_t>(g2, strategies); | ||
geometry::detail::expand_by_epsilon(b2); | ||
|
||
for (auto it = rtree.qbegin(index::nearest(b2, rtree.size())) ; it != rtree_qend ; ++it) |
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.
Minor, several lines are too long, I guess we still try to limit width up to 80 chars?
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.
We increased max to 100 some time ago. From Guidelines-for-Developers:
- The preferred line length is 80 characters, with maximum length of 100.
- The limit is relaxed for very long string literals (e.g. Well-Known Text with data used in tests and examples).
No description provided.