From: Jonathan Wakely <jwakely@redhat.com>
Date: Fri, 15 Jul 2016 19:51:33 +0000 (+0100)
Subject: Remove redundant std::move in std::for_each
X-Git-Tag: upstream/12.2.0~45832
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d1129d45b09c6fe0542f439fffda3bd4667a67a1;p=platform%2Fupstream%2Fgcc.git

Remove redundant std::move in std::for_each

	* include/bits/stl_algo.h (for_each): Remove redundant _GLIBCXX_MOVE
	and adjust comment.

From-SVN: r238399
---

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 3e4a0f5..f3ca34e 100644
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,5 +1,8 @@
 2016-07-15  Jonathan Wakely  <jwakely@redhat.com>
 
+	* include/bits/stl_algo.h (for_each): Remove redundant _GLIBCXX_MOVE
+	and adjust comment.
+
 	PR c++/58796
 	* libsupc++/pbase_type_info.cc (__pbase_type_info::__do_catch): Make
 	nullptr match handlers of pointer type.
diff --git a/libstdc++-v3/include/bits/stl_algo.h b/libstdc++-v3/include/bits/stl_algo.h
index a0820d4..ea0b56c 100644
--- a/libstdc++-v3/include/bits/stl_algo.h
+++ b/libstdc++-v3/include/bits/stl_algo.h
@@ -3791,7 +3791,7 @@ _GLIBCXX_BEGIN_NAMESPACE_ALGO
    *  @param  __first  An input iterator.
    *  @param  __last   An input iterator.
    *  @param  __f      A unary function object.
-   *  @return   @p __f (std::move(@p __f) in C++0x).
+   *  @return   @p __f
    *
    *  Applies the function object @p __f to each element in the range
    *  @p [first,last).  @p __f must not modify the order of the sequence.
@@ -3806,7 +3806,7 @@ _GLIBCXX_BEGIN_NAMESPACE_ALGO
       __glibcxx_requires_valid_range(__first, __last);
       for (; __first != __last; ++__first)
 	__f(*__first);
-      return _GLIBCXX_MOVE(__f);
+      return __f; // N.B. [alg.foreach] says std::move(f) but it's redundant.
     }
 
   /**