diff options
| author | Mike Bjorge <mbjorge@google.com> | 2017-01-09 16:02:55 -0800 |
|---|---|---|
| committer | Mike Bjorge <mbjorge@google.com> | 2017-01-09 16:02:55 -0800 |
| commit | 06a81e9357b6e5cf023ac65b7468191cd1949d42 (patch) | |
| tree | cbb1bebfc788fc467c982bd31df498f81846a9d9 | |
| parent | 5e7fd50e17b6edf1cadff973d0ec68966cf3265e (diff) | |
| download | googletest-06a81e9357b6e5cf023ac65b7468191cd1949d42.tar.gz googletest-06a81e9357b6e5cf023ac65b7468191cd1949d42.tar.bz2 googletest-06a81e9357b6e5cf023ac65b7468191cd1949d42.zip | |
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.
| -rw-r--r-- | googletest/include/gtest/gtest-typed-test.h | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/googletest/include/gtest/gtest-typed-test.h b/googletest/include/gtest/gtest-typed-test.h index 5f69d567..f4be9e55 100644 --- 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) = \ - GTEST_TYPED_TEST_CASE_P_STATE_(CaseName).VerifyRegisteredTestNames(\ - __FILE__, __LINE__, #__VA_ARGS__) + static const char* const GTEST_REGISTERED_TEST_NAMES_(CaseName) \ + GTEST_ATTRIBUTE_UNUSED = \ + 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 |