diff options
| author | kosak <kosak@google.com> | 2014-06-18 00:22:42 +0000 | 
|---|---|---|
| committer | kosak <kosak@google.com> | 2014-06-18 00:22:42 +0000 | 
| commit | 96ddffe8fdabccb10fb693a54dcb88bd5b71bc09 (patch) | |
| tree | ce5d8e5647e5c70cd0b052ad548ae94ae20431bb /src | |
| parent | 21ee8a2e72871ca50148657bc217e9b838dcd903 (diff) | |
| download | googletest-96ddffe8fdabccb10fb693a54dcb88bd5b71bc09.tar.gz googletest-96ddffe8fdabccb10fb693a54dcb88bd5b71bc09.tar.bz2 googletest-96ddffe8fdabccb10fb693a54dcb88bd5b71bc09.zip | |
Reduce the number of occurrences of gendered pronouns in gtest.
Diffstat (limited to 'src')
| -rw-r--r-- | src/gtest-death-test.cc | 6 | ||||
| -rw-r--r-- | src/gtest-internal-inl.h | 2 | ||||
| -rw-r--r-- | src/gtest-port.cc | 6 | ||||
| -rw-r--r-- | src/gtest-test-part.cc | 6 | ||||
| -rw-r--r-- | src/gtest.cc | 8 | 
5 files changed, 14 insertions, 14 deletions
| diff --git a/src/gtest-death-test.cc b/src/gtest-death-test.cc index 69fe7c1b..a0a8c7ba 100644 --- a/src/gtest-death-test.cc +++ b/src/gtest-death-test.cc @@ -68,9 +68,9 @@  // Indicates that this translation unit is part of Google Test's  // implementation.  It must come before gtest-internal-inl.h is -// included, or there will be a compiler error.  This trick is to -// prevent a user from accidentally including gtest-internal-inl.h in -// his code. +// included, or there will be a compiler error.  This trick exists to +// prevent the accidental inclusion of gtest-internal-inl.h in the +// user's code.  #define GTEST_IMPLEMENTATION_ 1  #include "src/gtest-internal-inl.h"  #undef GTEST_IMPLEMENTATION_ diff --git a/src/gtest-internal-inl.h b/src/gtest-internal-inl.h index 41fadb10..0ac7a109 100644 --- a/src/gtest-internal-inl.h +++ b/src/gtest-internal-inl.h @@ -40,7 +40,7 @@  // GTEST_IMPLEMENTATION_ is defined to 1 iff the current translation unit is  // part of Google Test's implementation; otherwise it's undefined.  #if !GTEST_IMPLEMENTATION_ -// A user is trying to include this from his code - just say no. +// If this file is included from the user's code, just say no.  # error "gtest-internal-inl.h is part of Google Test's internal implementation."  # error "It must not be included except by Google Test itself."  #endif  // GTEST_IMPLEMENTATION_ diff --git a/src/gtest-port.cc b/src/gtest-port.cc index 39e70bb0..b032745b 100644 --- a/src/gtest-port.cc +++ b/src/gtest-port.cc @@ -64,9 +64,9 @@  // Indicates that this translation unit is part of Google Test's  // implementation.  It must come before gtest-internal-inl.h is -// included, or there will be a compiler error.  This trick is to -// prevent a user from accidentally including gtest-internal-inl.h in -// his code. +// included, or there will be a compiler error.  This trick exists to +// prevent the accidental inclusion of gtest-internal-inl.h in the +// user's code.  #define GTEST_IMPLEMENTATION_ 1  #include "src/gtest-internal-inl.h"  #undef GTEST_IMPLEMENTATION_ diff --git a/src/gtest-test-part.cc b/src/gtest-test-part.cc index c60eef3a..fb0e3542 100644 --- a/src/gtest-test-part.cc +++ b/src/gtest-test-part.cc @@ -35,9 +35,9 @@  // Indicates that this translation unit is part of Google Test's  // implementation.  It must come before gtest-internal-inl.h is -// included, or there will be a compiler error.  This trick is to -// prevent a user from accidentally including gtest-internal-inl.h in -// his code. +// included, or there will be a compiler error.  This trick exists to +// prevent the accidental inclusion of gtest-internal-inl.h in the +// user's code.  #define GTEST_IMPLEMENTATION_ 1  #include "src/gtest-internal-inl.h"  #undef GTEST_IMPLEMENTATION_ diff --git a/src/gtest.cc b/src/gtest.cc index 408e6f2c..ca3596eb 100644 --- a/src/gtest.cc +++ b/src/gtest.cc @@ -1962,8 +1962,8 @@ bool Test::HasSameFixtureClass() {      const bool this_is_TEST = this_fixture_id == internal::GetTestTypeId();      if (first_is_TEST || this_is_TEST) { -      // The user mixed TEST and TEST_F in this test case - we'll tell -      // him/her how to fix it. +      // Both TEST and TEST_F appear in same test case, which is incorrect. +      // Tell the user how to fix this.        // Gets the name of the TEST and the name of the TEST_F.  Note        // that first_is_TEST and this_is_TEST cannot both be true, as @@ -1983,8 +1983,8 @@ bool Test::HasSameFixtureClass() {            << "want to change the TEST to TEST_F or move it to another test\n"            << "case.";      } else { -      // The user defined two fixture classes with the same name in -      // two namespaces - we'll tell him/her how to fix it. +      // Two fixture classes with the same name appear in two different +      // namespaces, which is not allowed. Tell the user how to fix this.        ADD_FAILURE()            << "All tests in the same test case must use the same test fixture\n"            << "class.  However, in test case " | 
