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 /googletest/include/gtest/gtest-matchers.h | |
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 'googletest/include/gtest/gtest-matchers.h')
-rw-r--r-- | googletest/include/gtest/gtest-matchers.h | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/googletest/include/gtest/gtest-matchers.h b/googletest/include/gtest/gtest-matchers.h index bf500fc..6e73ba1 100644 --- a/googletest/include/gtest/gtest-matchers.h +++ b/googletest/include/gtest/gtest-matchers.h @@ -47,9 +47,16 @@ #include "gtest/internal/gtest-internal.h" #include "gtest/internal/gtest-port.h" +// MSVC warning C5046 is new as of VS2017 version 15.8. +#if defined(_MSC_VER) && _MSC_VER >= 1915 +#define GTEST_MAYBE_5046_ 5046 +#else +#define GTEST_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 GTEST_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 { |