diff options
author | vladlosev <vladlosev@861a406c-534a-0410-8894-cb66d6ee9925> | 2011-11-04 17:56:23 +0000 |
---|---|---|
committer | vladlosev <vladlosev@861a406c-534a-0410-8894-cb66d6ee9925> | 2011-11-04 17:56:23 +0000 |
commit | 8965a6a0d2165f32e6413594bba6367f271f51e7 (patch) | |
tree | ab0e0f5e0fd23697673ec2e0edca67079375dd2c /test/gtest-port_test.cc | |
parent | 829402edcffe712ed4c79412ca020525cd8295ad (diff) | |
download | googletest-8965a6a0d2165f32e6413594bba6367f271f51e7.tar.gz googletest-8965a6a0d2165f32e6413594bba6367f271f51e7.tar.bz2 googletest-8965a6a0d2165f32e6413594bba6367f271f51e7.zip |
Improves conformance to the Google C++ Style Guide (by Greg Miller).
Diffstat (limited to 'test/gtest-port_test.cc')
-rw-r--r-- | test/gtest-port_test.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/gtest-port_test.cc b/test/gtest-port_test.cc index b0177cf1..75471c39 100644 --- a/test/gtest-port_test.cc +++ b/test/gtest-port_test.cc @@ -92,7 +92,7 @@ TEST(ImplicitCastTest, CanUseInheritance) { class Castable { public: - Castable(bool* converted) : converted_(converted) {} + explicit Castable(bool* converted) : converted_(converted) {} operator Base() { *converted_ = true; return Base(); @@ -111,7 +111,7 @@ TEST(ImplicitCastTest, CanUseNonConstCastOperator) { class ConstCastable { public: - ConstCastable(bool* converted) : converted_(converted) {} + explicit ConstCastable(bool* converted) : converted_(converted) {} operator Base() const { *converted_ = true; return Base(); @@ -224,7 +224,7 @@ TEST(GtestCheckSyntaxTest, WorksWithSwitch) { GTEST_CHECK_(true); } - switch(0) + switch (0) case 0: GTEST_CHECK_(true) << "Check failed in switch case"; } @@ -929,7 +929,7 @@ TEST(CaptureTest, CapturesStdoutAndStderr) { TEST(CaptureDeathTest, CannotReenterStdoutCapture) { CaptureStdout(); - EXPECT_DEATH_IF_SUPPORTED(CaptureStdout();, + EXPECT_DEATH_IF_SUPPORTED(CaptureStdout(), "Only one stdout capturer can exist at a time"); GetCapturedStdout(); |