You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bb...@apache.org on 2019/01/04 18:53:37 UTC

[geode-native] branch develop updated: GEODE-6245: Fix Microsoft pragma warnings (#429)

This is an automated email from the ASF dual-hosted git repository.

bbender pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
     new d9a64c6  GEODE-6245: Fix Microsoft pragma warnings (#429)
d9a64c6 is described below

commit d9a64c637ae7e4edebe9e8edc020b164f74ae917
Author: Sai Boorlagadda <sa...@gmail.com>
AuthorDate: Fri Jan 4 10:53:32 2019 -0800

    GEODE-6245: Fix Microsoft pragma warnings (#429)
    
    - Caused "unknown pragma" warnings when building via gcc
    - Only use in presence of MS toolchain
    
    Co-authored-by: Matthew Reddington <mr...@pivotal.io>
---
 cppcache/include/geode/Exception.hpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/cppcache/include/geode/Exception.hpp b/cppcache/include/geode/Exception.hpp
index 2df6d3f..8bde954 100644
--- a/cppcache/include/geode/Exception.hpp
+++ b/cppcache/include/geode/Exception.hpp
@@ -33,9 +33,11 @@ namespace client {
 
 class StackTrace;
 
+#if defined(_MSC_VER)
 // Ignore C4275 - This class extends std C++ class
 #pragma warning(push)
 #pragma warning(disable : 4275)
+#endif
 
 /**
  * A description of an exception that occurred during a cache operation.
@@ -73,7 +75,9 @@ class APACHE_GEODE_EXPORT Exception : public std::exception {
   std::shared_ptr<StackTrace> stack_;
 };
 
+#if defined(_MSC_VER)
 #pragma warning(pop)
+#endif
 
 }  // namespace client
 }  // namespace geode