more comments changes
This commit is contained in:
parent
265efde9a5
commit
bbf738a2c1
googlemock/src
googletest/test
|
@ -606,7 +606,7 @@ class MockObjectRegistry {
|
|||
if (it->second.leakable) // The user said it's fine to leak this object.
|
||||
continue;
|
||||
|
||||
// TODO(wan@google.com): Print the type of the leaked object.
|
||||
// FIXME: Print the type of the leaked object.
|
||||
// This can help the user identify the leaked object.
|
||||
std::cout << "\n";
|
||||
const MockObjectState& state = it->second;
|
||||
|
|
|
@ -33,7 +33,7 @@
|
|||
|
||||
namespace testing {
|
||||
|
||||
// TODO(wan@google.com): support using environment variables to
|
||||
// FIXME: support using environment variables to
|
||||
// control the flag values, like what Google Test does.
|
||||
|
||||
GMOCK_DEFINE_bool_(catch_leaked_mocks, true,
|
||||
|
|
|
@ -106,7 +106,7 @@ TEST(OutputFileHelpersTest, GetCurrentExecutableName) {
|
|||
const bool success = exe_str == "app";
|
||||
#else
|
||||
// FIXME: remove the hard-coded "lt-" prefix when
|
||||
// Chandler Carruth's libtool replacement is ready.
|
||||
// libtool replacement is ready.
|
||||
const bool success =
|
||||
exe_str == "googletest-options-test" ||
|
||||
exe_str == "gtest_all_test" ||
|
||||
|
|
|
@ -118,7 +118,7 @@ class MyParamTest : public testing::TestWithParam<int> {};
|
|||
|
||||
TEST_P(MyParamTest, ShouldPass) {
|
||||
// FIXME: Make parameter value checking robust
|
||||
// WRT order of tests.
|
||||
// WRT order of tests.
|
||||
GTEST_CHECK_INT_EQ_(g_param_test_count % kNumberOfParamTests, GetParam());
|
||||
g_param_test_count++;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue
Block a user