-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Replace boost by std #7403
Replace boost by std #7403
Conversation
boost's version is working on std::pair but we decided to no longer consider iterator pairs as ranges
boost::is_float
Replace boost by std
b36bf0a
to
e80e05b
Compare
e80e05b
to
dfc386d
Compare
@@ -154,11 +155,11 @@ class Polyline_simplification_2 | |||
(*it)->set_removable(false); | |||
++it; | |||
for(; it != ite; ++it){ | |||
if((std::next(it) != ite) && (std::prev(it)== std::next(it))){ | |||
if((boost::next(it) != ite) && (boost::prior(it)==boost::prior(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.
Any idea why the behavior is different?
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.
The iterator implementation is most probably buggy (some platforms complains that it is not bidirectional and there are infinite loops on other platforms).
…stacking boost::iterator_adaptor also restore std::prev
Successfully tested in |
Summary of Changes
Replace features in boost by features introduces in c++11/c++14