From e283930566dc8fc5bfb410df3269e921933b3978 Mon Sep 17 00:00:00 2001
From: Victor Zverovich <victor.zverovich@gmail.com>
Date: Tue, 6 May 2014 12:13:16 -0700
Subject: [PATCH] Fix warnings.

---
 test/gtest-extra-test.cc | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/test/gtest-extra-test.cc b/test/gtest-extra-test.cc
index 53461321..0c5d1056 100644
--- a/test/gtest-extra-test.cc
+++ b/test/gtest-extra-test.cc
@@ -39,10 +39,11 @@
 
 namespace {
 
+#ifdef _WIN32
+
 // Suppresses Windows assertions on invalid file descriptors, making
 // POSIX functions return proper error codes instead of crashing on Windows.
 class SuppressAssert {
-#ifdef _WIN32
  private:
   _invalid_parameter_handler original_handler_;
   int original_report_mode_;
@@ -59,10 +60,12 @@ class SuppressAssert {
     _set_invalid_parameter_handler(original_handler_);
     _CrtSetReportMode(_CRT_ASSERT, original_report_mode_);
   }
-#endif  // _WIN32
 };
 
-#define SUPPRESS_ASSERT(statement) { SuppressAssert sa; statement; }
+# define SUPPRESS_ASSERT(statement) { SuppressAssert sa; statement; }
+#else
+# define SUPPRESS_ASSERT(statement) statement
+#endif  // _WIN32
 
 #define EXPECT_SYSTEM_ERROR_NOASSERT(statement, error_code, message) \
   EXPECT_SYSTEM_ERROR(SUPPRESS_ASSERT(statement), error_code, message)
@@ -182,7 +185,7 @@ TEST(AssertionSyntaxTest, ExceptionAssertionBehavesLikeSingleStatement) {
   if (::testing::internal::AlwaysTrue())
     EXPECT_THROW_MSG(ThrowException(), std::exception, "test");
   else
-    ;  // NOLINT
+    DoNothing();
 }
 
 TEST(AssertionSyntaxTest, WriteAssertionBehavesLikeSingleStatement) {
@@ -192,7 +195,7 @@ TEST(AssertionSyntaxTest, WriteAssertionBehavesLikeSingleStatement) {
   if (::testing::internal::AlwaysTrue())
     EXPECT_WRITE(stdout, std::printf("x"), "x");
   else
-    ;  // NOLINT
+    DoNothing();
 }
 
 // Tests EXPECT_THROW_MSG.