Re: [PATCH] Remove redundant std::move in std::for_each

2016-07-15 Thread Daniel Krügler
2016-07-15 21:53 GMT+02:00 Jonathan Wakely : > Returning a parameter will try to move anyway, so using std::move here > is redundant (and clang even warns about it being redundant). > > * include/bits/stl_algo.h (for_each): Remove redundant _GLIBCXX_MOVE > and adjust comment. Actua

[PATCH] Remove redundant std::move in std::for_each

2016-07-15 Thread Jonathan Wakely
Returning a parameter will try to move anyway, so using std::move here is redundant (and clang even warns about it being redundant). * include/bits/stl_algo.h (for_each): Remove redundant _GLIBCXX_MOVE and adjust comment. Tested x86_64-linux, committed to trunk. commit 54ee8abfe