Add GTEST_ATTRIBUTE_UNUSED_ to REGISTER_TYPED_TEST_CASE_P

If REGISTER_TYPED_TEST_CASE_P is included in a header file, but the .cc file does not declare INSTANTIATE_TYPED_TEST_CASE_P, an unused-variable warning may be raised by the compiler.

Add GTEST_ATTRIBUTE_UNUSED_ to REGISTER_TYPED_TEST_CASE_P
06a81e93 · Mike Bjorge · 5e7fd50e · 06a81e93
Commit 06a81e93 authored Jan 09, 2017 by Mike Bjorge
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 3 deletions

gtest-typed-test.h googletest/include/gtest/gtest-typed-test.h +4 -3

No files found.
--- a/googletest/include/gtest/gtest-typed-test.h
+++ b/googletest/include/gtest/gtest-typed-test.h
@@ -241,9 +241,10 @@ INSTANTIATE_TYPED_TEST_CASE_P(My, FooTest, MyTypes);
  namespace GTEST_CASE_NAMESPACE_(CaseName) { \
  typedef ::testing::internal::Templates<__VA_ARGS__>::type gtest_AllTests_; \
  } \
-  static const char* const GTEST_REGISTERED_TEST_NAMES_(CaseName) = \
+  static const char* const GTEST_REGISTERED_TEST_NAMES_(CaseName) \
-      GTEST_TYPED_TEST_CASE_P_STATE_(CaseName).VerifyRegisteredTestNames(\
+      GTEST_ATTRIBUTE_UNUSED = \
-          __FILE__, __LINE__, #__VA_ARGS__)
+          GTEST_TYPED_TEST_CASE_P_STATE_(CaseName).VerifyRegisteredTestNames(\
+              __FILE__, __LINE__, #__VA_ARGS__)
 // The 'Types' template argument below must have spaces around it
 // since some compilers may choke on '>>' when passing a template