From 109254f421d5837465a3eff104b6774dcf33c933 Mon Sep 17 00:00:00 2001 From: varconst Date: Wed, 17 Mar 2021 19:34:10 -0400 Subject: [PATCH] Update comment --- googletest/include/gtest/gtest.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/googletest/include/gtest/gtest.h b/googletest/include/gtest/gtest.h index 1088e110..bc4d73e7 100644 --- a/googletest/include/gtest/gtest.h +++ b/googletest/include/gtest/gtest.h @@ -1992,11 +1992,11 @@ class TestWithParam : public Test, public WithParamInterface { // Note: // // 1. It is possible to make a user-defined type work with -// {ASSERT|EXPECT}_??(), but that requires overloading the -// comparison operators and is thus discouraged by the Google C++ -// Usage Guide. Therefore, you are advised to use the -// {ASSERT|EXPECT}_TRUE() macro to assert that two objects are -// equal. +// {ASSERT|EXPECT}_??() by overloading the comparison operators. Note +// that these macros rely on argument-dependent lookup to find the +// comparison operators, so the operators must be defined in the same +// namespace as the types being compared (or at least one of the +// types if they are from different namespaces). // // 2. The {ASSERT|EXPECT}_??() macros do pointer comparisons on // pointers (in particular, C strings). Therefore, if you use it