diff options
| author | David Benjamin <davidben@google.com> | 2019-04-09 22:55:28 (GMT) |
|---|---|---|
| committer | David Benjamin <davidben@google.com> | 2019-04-09 23:13:32 (GMT) |
| commit | 8e9297b503d6206b16389c756fde80f2cb2db6f4 (patch) | |
| tree | f37760a6f4cf72aee314845c95e4f0b789ba2ebf /googlemock | |
| parent | 604ba376c3a407c6a40e39fbd0d5055c545f9898 (diff) | |
| download | googletest-8e9297b503d6206b16389c756fde80f2cb2db6f4.zip googletest-8e9297b503d6206b16389c756fde80f2cb2db6f4.tar.gz googletest-8e9297b503d6206b16389c756fde80f2cb2db6f4.tar.bz2 | |
MSVC C5046 warning is unavailable in MSVC 2015.refs/pull/2226/head
Per the MSVC documentation the warning is new as of Visual Studio 2017,
version 15.8.
https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/c5046?view=vs-2019
GTest users building on MSVC 2015 or older versions of 2017 will, when
C4616 is enabled, see a warning like:
[...]gtest-matchers.h(53): error C2220: warning treated as error - no 'object' file generated
[...]gtest-matchers.h(53): warning C4619: #pragma warning: there is no warning number '5046'
Guard the mention of 5046 by an _MSC_VER check. VS2017 15.8 corresponds
to an _MSC_VER of 1915.
https://docs.microsoft.com/en-us/cpp/preprocessor/predefined-macros?view=vs-2019
Diffstat (limited to 'googlemock')
| -rw-r--r-- | googlemock/include/gmock/gmock-matchers.h | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/googlemock/include/gmock/gmock-matchers.h b/googlemock/include/gmock/gmock-matchers.h index 3e4ee40..fcf8cf2 100644 --- a/googlemock/include/gmock/gmock-matchers.h +++ b/googlemock/include/gmock/gmock-matchers.h @@ -58,9 +58,16 @@ #include "gmock/internal/gmock-port.h" #include "gtest/gtest.h" +// MSVC warning C5046 is new as of VS2017 version 15.8. +#if defined(_MSC_VER) && _MSC_VER >= 1915 +#define GMOCK_MAYBE_5046_ 5046 +#else +#define GMOCK_MAYBE_5046_ +#endif + GTEST_DISABLE_MSC_WARNINGS_PUSH_( - 4251 5046 /* class A needs to have dll-interface to be used by clients of - class B */ + 4251 GMOCK_MAYBE_5046_ /* class A needs to have dll-interface to be used by + clients of class B */ /* Symbol involving type with internal linkage not defined */) namespace testing { |