Googletest export

Fix std::move to std::forward where appropriate to support reference types.

PiperOrigin-RevId: 292923058

4 files changed, 68 insertions, 34 deletions

diff --git a/googlemock/include/gmock/gmock-generated-matchers.h b/googlemock/include/gmock/gmock-generated-matchers.h
index 539d657..c9dc552 100644
--- a/googlemock/include/gmock/gmock-generated-matchers.h
+++ b/googlemock/include/gmock/gmock-generated-matchers.h
@@ -100,7 +100,7 @@
         GTEST_REFERENCE_TO_CONST_(arg_type)> {\
      public:\
       explicit gmock_Impl(p0##_type gmock_p0)\
-           : p0(::std::move(gmock_p0)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -147,7 +147,8 @@
         GTEST_REFERENCE_TO_CONST_(arg_type)> {\
      public:\
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -197,8 +198,9 @@
         GTEST_REFERENCE_TO_CONST_(arg_type)> {\
      public:\
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -251,8 +253,10 @@
      public:\
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -311,9 +315,11 @@
      public:\
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3, p4##_type gmock_p4)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -373,9 +379,12 @@
      public:\
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3, p4##_type gmock_p4, p5##_type gmock_p5)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)), p5(::std::move(gmock_p5)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)), \
+               p5(::std::forward<p5##_type>(gmock_p5)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -439,10 +448,13 @@
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3, p4##_type gmock_p4, p5##_type gmock_p5, \
           p6##_type gmock_p6)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)), p5(::std::move(gmock_p5)), \
-               p6(::std::move(gmock_p6)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)), \
+               p5(::std::forward<p5##_type>(gmock_p5)), \
+               p6(::std::forward<p6##_type>(gmock_p6)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -512,10 +524,14 @@
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3, p4##_type gmock_p4, p5##_type gmock_p5, \
           p6##_type gmock_p6, p7##_type gmock_p7)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)), p5(::std::move(gmock_p5)), \
-               p6(::std::move(gmock_p6)), p7(::std::move(gmock_p7)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)), \
+               p5(::std::forward<p5##_type>(gmock_p5)), \
+               p6(::std::forward<p6##_type>(gmock_p6)), \
+               p7(::std::forward<p7##_type>(gmock_p7)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -588,11 +604,15 @@
       gmock_Impl(p0##_type gmock_p0, p1##_type gmock_p1, p2##_type gmock_p2, \
           p3##_type gmock_p3, p4##_type gmock_p4, p5##_type gmock_p5, \
           p6##_type gmock_p6, p7##_type gmock_p7, p8##_type gmock_p8)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)), p5(::std::move(gmock_p5)), \
-               p6(::std::move(gmock_p6)), p7(::std::move(gmock_p7)), \
-               p8(::std::move(gmock_p8)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)), \
+               p5(::std::forward<p5##_type>(gmock_p5)), \
+               p6(::std::forward<p6##_type>(gmock_p6)), \
+               p7(::std::forward<p7##_type>(gmock_p7)), \
+               p8(::std::forward<p8##_type>(gmock_p8)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
@@ -669,11 +689,16 @@
           p3##_type gmock_p3, p4##_type gmock_p4, p5##_type gmock_p5, \
           p6##_type gmock_p6, p7##_type gmock_p7, p8##_type gmock_p8, \
           p9##_type gmock_p9)\
-           : p0(::std::move(gmock_p0)), p1(::std::move(gmock_p1)), \
-               p2(::std::move(gmock_p2)), p3(::std::move(gmock_p3)), \
-               p4(::std::move(gmock_p4)), p5(::std::move(gmock_p5)), \
-               p6(::std::move(gmock_p6)), p7(::std::move(gmock_p7)), \
-               p8(::std::move(gmock_p8)), p9(::std::move(gmock_p9)) {}\
+           : p0(::std::forward<p0##_type>(gmock_p0)), \
+               p1(::std::forward<p1##_type>(gmock_p1)), \
+               p2(::std::forward<p2##_type>(gmock_p2)), \
+               p3(::std::forward<p3##_type>(gmock_p3)), \
+               p4(::std::forward<p4##_type>(gmock_p4)), \
+               p5(::std::forward<p5##_type>(gmock_p5)), \
+               p6(::std::forward<p6##_type>(gmock_p6)), \
+               p7(::std::forward<p7##_type>(gmock_p7)), \
+               p8(::std::forward<p8##_type>(gmock_p8)), \
+               p9(::std::forward<p9##_type>(gmock_p9)) {}\
       bool MatchAndExplain(\
           GTEST_REFERENCE_TO_CONST_(arg_type) arg,\
           ::testing::MatchResultListener* result_listener) const override;\
diff --git a/googlemock/include/gmock/gmock-generated-matchers.h.pump b/googlemock/include/gmock/gmock-generated-matchers.h.pump
index 0ea3aad..d72daa8 100644
--- a/googlemock/include/gmock/gmock-generated-matchers.h.pump
+++ b/googlemock/include/gmock/gmock-generated-matchers.h.pump
@@ -63,7 +63,7 @@ $var template = [[$if i==0 [[]] $else [[
   template <$for j, [[typename p$j##_type]]>\
 ]]]]
 $var impl_ctor_param_list = [[$for j, [[p$j##_type gmock_p$j]]]]
-$var impl_inits = [[$if i==0 [[]] $else [[ : $for j, [[p$j(::std::move(gmock_p$j))]]]]]]
+$var impl_inits = [[$if i==0 [[]] $else [[ : $for j, [[p$j(::std::forward<p$j##_type>(gmock_p$j))]]]]]]
 $var params = [[$for j, [[p$j]]]]
 $var param_types = [[$if i==0 [[]] $else [[<$for j, [[p$j##_type]]>]]]]
 $var param_types_and_names = [[$for j, [[p$j##_type p$j]]]]
diff --git a/googlemock/include/gmock/gmock-matchers.h b/googlemock/include/gmock/gmock-matchers.h
index 473b3aa..7d136af 100644
--- a/googlemock/include/gmock/gmock-matchers.h
+++ b/googlemock/include/gmock/gmock-matchers.h
@@ -470,10 +470,12 @@ class MatcherBaseImpl<Derived<Ts...>> {
   // conversions.
   template <typename E = std::enable_if<sizeof...(Ts) == 1>,
             typename E::type* = nullptr>
-  explicit MatcherBaseImpl(Ts... params) : params_(std::move(params)...) {}
+  explicit MatcherBaseImpl(Ts... params)
+      : params_(std::forward<Ts>(params)...) {}
   template <typename E = std::enable_if<sizeof...(Ts) != 1>,
             typename = typename E::type>
-  MatcherBaseImpl(Ts... params) : params_(std::move(params)...) {}  // NOLINT
+  MatcherBaseImpl(Ts... params)  // NOLINT
+      : params_(std::forward<Ts>(params)...) {}
 
   template <typename F>
   operator ::testing::Matcher<F>() const {  // NOLINT(runtime/explicit)
diff --git a/googlemock/test/gmock-generated-matchers_test.cc b/googlemock/test/gmock-generated-matchers_test.cc
index 6783f8f..a5ab4e2 100644
--- a/googlemock/test/gmock-generated-matchers_test.cc
+++ b/googlemock/test/gmock-generated-matchers_test.cc
@@ -764,9 +764,16 @@ MATCHER_P2(ReferencesAnyOf, variable1, variable2, "") {
 
 TEST(MatcherPnMacroTest, WorksWhenExplicitlyInstantiatedWithReferences) {
   UncopyableFoo foo1('1'), foo2('2'), foo3('3');
-  const Matcher<const UncopyableFoo&> m =
+  const Matcher<const UncopyableFoo&> const_m =
       ReferencesAnyOf<const UncopyableFoo&, const UncopyableFoo&>(foo1, foo2);
 
+  EXPECT_TRUE(const_m.Matches(foo1));
+  EXPECT_TRUE(const_m.Matches(foo2));
+  EXPECT_FALSE(const_m.Matches(foo3));
+
+  const Matcher<UncopyableFoo&> m =
+      ReferencesAnyOf<UncopyableFoo&, UncopyableFoo&>(foo1, foo2);
+
   EXPECT_TRUE(m.Matches(foo1));
   EXPECT_TRUE(m.Matches(foo2));
   EXPECT_FALSE(m.Matches(foo3));