diff options
| author | Matt Calabrese <calabrese@x.team> | 2019-12-13 16:35:50 (GMT) |
|---|---|---|
| committer | Matt Calabrese <calabrese@x.team> | 2019-12-13 16:35:50 (GMT) |
| commit | 3482353e534b894873ac7901489f8dd469759ce6 (patch) | |
| tree | 75ef3d7ad08ba72928e8708a596e9fd791c7a44d | |
| parent | 78ebd4a79615183beec807953d6aead06ce824f5 (diff) | |
| parent | e1dd49835ef24a70897286dd4bf79df186af596a (diff) | |
| download | googletest-3482353e534b894873ac7901489f8dd469759ce6.zip googletest-3482353e534b894873ac7901489f8dd469759ce6.tar.gz googletest-3482353e534b894873ac7901489f8dd469759ce6.tar.bz2 | |
Merge pull request #2589 from kuzkry:remove-workaround_g++-no-space-after-first-macro-argumentrefs/pull/2633/head1278977C598F83A02A525FEA38941265
PiperOrigin-RevId: 285255373
| -rw-r--r-- | googletest/test/gtest_unittest.cc | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/googletest/test/gtest_unittest.cc b/googletest/test/gtest_unittest.cc index d447d1b..7acc228 100644 --- a/googletest/test/gtest_unittest.cc +++ b/googletest/test/gtest_unittest.cc @@ -2923,22 +2923,18 @@ TEST_F(FloatTest, Commutative) { TEST_F(FloatTest, EXPECT_NEAR) { EXPECT_NEAR(-1.0f, -1.1f, 0.2f); EXPECT_NEAR(2.0f, 3.0f, 1.0f); - EXPECT_NONFATAL_FAILURE(EXPECT_NEAR(1.0f,1.5f, 0.25f), // NOLINT + EXPECT_NONFATAL_FAILURE(EXPECT_NEAR(1.0f, 1.5f, 0.25f), // NOLINT "The difference between 1.0f and 1.5f is 0.5, " "which exceeds 0.25f"); - // To work around a bug in gcc 2.95.0, there is intentionally no - // space after the first comma in the previous line. } // Tests ASSERT_NEAR. TEST_F(FloatTest, ASSERT_NEAR) { ASSERT_NEAR(-1.0f, -1.1f, 0.2f); ASSERT_NEAR(2.0f, 3.0f, 1.0f); - EXPECT_FATAL_FAILURE(ASSERT_NEAR(1.0f,1.5f, 0.25f), // NOLINT + EXPECT_FATAL_FAILURE(ASSERT_NEAR(1.0f, 1.5f, 0.25f), // NOLINT "The difference between 1.0f and 1.5f is 0.5, " "which exceeds 0.25f"); - // To work around a bug in gcc 2.95.0, there is intentionally no - // space after the first comma in the previous line. } // Tests the cases where FloatLE() should succeed. @@ -4321,10 +4317,8 @@ TEST(AssertionWithMessageTest, ASSERT_STR) { TEST(AssertionWithMessageTest, ASSERT_FLOATING) { ASSERT_FLOAT_EQ(1, 1) << "This should succeed."; ASSERT_DOUBLE_EQ(1, 1) << "This should succeed."; - EXPECT_FATAL_FAILURE(ASSERT_NEAR(1,1.2, 0.1) << "Expect failure.", // NOLINT + EXPECT_FATAL_FAILURE(ASSERT_NEAR(1, 1.2, 0.1) << "Expect failure.", // NOLINT "Expect failure."); - // To work around a bug in gcc 2.95.0, there is intentionally no - // space after the first comma in the previous statement. } // Tests using ASSERT_FALSE with a streamed message. |