-
Notifications
You must be signed in to change notification settings - Fork 117
Add move_while and move_until new algorithms
#109
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
Conversation
denzor200
commented
Oct 24, 2022
CI is broken, so this PR tested here: https://godbolt.org/z/b4vGMnK55
jgopel
commented
Oct 24, 2022
question: Forgive my ignorance - why not just use a std::move_iterator? What does this accomplish that that doesn't already do?
denzor200
commented
Oct 25, 2022
why not just use a std::move_iterator?
I had same question for std::move, but it exists. Look:
std::move(v.begin(), v.end(), std::back_inserter(l));
does the same result as:
std::copy(std::make_move_iterator(v.begin()),
std::make_move_iterator(v.end()),
std::back_inserter(l));
What does this accomplish that that doesn't already do?
just readability, and those people who use boost::algorithm::copy_while and boost::algorithm::copy_until also need for move_while and move_until.
Hello, i have an interest to be contributor of this library. I decided to start with this small PR to get experience and to receive feedback and "ping response" from community.