pdxcodemonkey commented on a change in pull request #741:
URL: https://github.com/apache/geode-native/pull/741#discussion_r571219081
##########
File path: cppcache/integration/benchmark/RegionBM.cpp
##########
@@ -19,6 +19,17 @@
#include <framework/Cluster.h>
#include <framework/Gfsh.h>
+// Disable warning for "extra qualifications" here. One of the boost log
+// headers triggers this warning. Note: use of disable pragma here is
+// intentional - attempts to use push/pop as you ordinarily should just
+// yielded a gripe from the MS tools that "warning number '4596' is not a
+// valid compiler warning". re-enabling the warning after the include
+// fails in the same way, so just leave it disabled for the rest of the
+// file. This is safe, since the warning can only trigger inside a class
Review comment:
What I'm saying is there's _no_ class defined in this file, cause that's
a thing typically done in headers. Thus, leaving the warning disabled for the
rest of the .cpp file has no effect.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
