From 1c22797cd3046a9fb74dc49268ac18ab86c14568 Mon Sep 17 00:00:00 2001 From: Enji Cooper Date: Wed, 13 Feb 2019 10:31:35 -0800 Subject: [PATCH] Fix clang `-Wunused-parameter` warnings Comment out unused method parameters in their relevant definitions. Signed-off-by: Enji Cooper --- googlemock/test/gmock-generated-matchers_test.cc | 2 +- googlemock/test/gmock-matchers_test.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/googlemock/test/gmock-generated-matchers_test.cc b/googlemock/test/gmock-generated-matchers_test.cc index eb47f2b0..6c4b3000 100644 --- a/googlemock/test/gmock-generated-matchers_test.cc +++ b/googlemock/test/gmock-generated-matchers_test.cc @@ -1262,7 +1262,7 @@ namespace adl_test { MATCHER(M, "") { return true; } template -bool AllOf(const T1& t1, const T2& t2) { return true; } +bool AllOf(const T1& /*t1*/, const T2& /*t2*/) { return true; } TEST(AllOfTest, DoesNotCallAllOfUnqualified) { EXPECT_THAT(42, testing::AllOf( diff --git a/googlemock/test/gmock-matchers_test.cc b/googlemock/test/gmock-matchers_test.cc index b586d276..40ccaf05 100644 --- a/googlemock/test/gmock-matchers_test.cc +++ b/googlemock/test/gmock-matchers_test.cc @@ -6943,7 +6943,7 @@ TEST(ArgsTest, ExplainsMatchResultWithoutInnerExplanation) { // For testing Args<>'s explanation. class LessThanMatcher : public MatcherInterface > { public: - void DescribeTo(::std::ostream* os) const override {} + void DescribeTo(::std::ostream* /*os*/) const override {} bool MatchAndExplain(std::tuple value, MatchResultListener* listener) const override {