diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-05-04 15:20:29 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-05-04 15:20:29 -0700 |
commit | aadcd71ab29f588d4997ec25855f60f5866959f2 (patch) | |
tree | 40be7e600c2010ad0c3a4caf91098d44afb4c7ca /core/src/fxcrt/fx_basic_wstring_unittest.cpp | |
parent | 2585a6bbd7dca65c4ee3c197937de9b0026a5e98 (diff) | |
download | pdfium-aadcd71ab29f588d4997ec25855f60f5866959f2.tar.xz |
Fix issues with != and == in fx_basic_wstring
Part two. Fix same issue in wide strings as in their bytestring
counterparts.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1127753002
Diffstat (limited to 'core/src/fxcrt/fx_basic_wstring_unittest.cpp')
-rw-r--r-- | core/src/fxcrt/fx_basic_wstring_unittest.cpp | 21 |
1 files changed, 2 insertions, 19 deletions
diff --git a/core/src/fxcrt/fx_basic_wstring_unittest.cpp b/core/src/fxcrt/fx_basic_wstring_unittest.cpp index 16d71cb5b8..3b15006d65 100644 --- a/core/src/fxcrt/fx_basic_wstring_unittest.cpp +++ b/core/src/fxcrt/fx_basic_wstring_unittest.cpp @@ -83,11 +83,7 @@ TEST(fxcrt, WideStringOperatorEQ) { const wchar_t* c_string_same1 = L"hello"; ASSERT_TRUE(wide_string == c_string_same1); -#if 0 - // TODO(tsepez): Missing operator - there's a prototype but no actual - // implementation (at least we already removed implicit c_str() casting). ASSERT_TRUE(c_string_same1 == wide_string); -#endif const wchar_t* c_string1 = L"he"; const wchar_t* c_string2 = L"hellp"; @@ -95,12 +91,9 @@ TEST(fxcrt, WideStringOperatorEQ) { ASSERT_FALSE(wide_string == c_string1); ASSERT_FALSE(wide_string == c_string2); ASSERT_FALSE(wide_string == c_string3); -#if 0 - // See above TODO. ASSERT_FALSE(c_string1 == wide_string); ASSERT_FALSE(c_string2 == wide_string); ASSERT_FALSE(c_string3 == wide_string); -#endif } TEST(fxcrt, WideStringOperatorNE) { @@ -141,22 +134,18 @@ TEST(fxcrt, WideStringOperatorNE) { const wchar_t* c_string_same1 = L"hello"; ASSERT_FALSE(wide_string != c_string_same1); -#if 0 - // See above TODO. ASSERT_FALSE(c_string_same1 != wide_string); -#endif + const wchar_t* c_string1 = L"he"; const wchar_t* c_string2 = L"hellp"; const wchar_t* c_string3 = L"hellod"; ASSERT_TRUE(wide_string != c_string1); ASSERT_TRUE(wide_string != c_string2); ASSERT_TRUE(wide_string != c_string3); -#if 0 - // See above TODO. + ASSERT_TRUE(c_string1 != wide_string); ASSERT_TRUE(c_string2 != wide_string); ASSERT_TRUE(c_string3 != wide_string); -#endif } #define ByteStringLiteral(str) CFX_ByteString(FX_BSTRC(str)) @@ -256,8 +245,6 @@ TEST(fxcrt, WideStringCOperatorEQ) { ASSERT_FALSE(wide_string2 == wide_string_c); ASSERT_FALSE(wide_string3 == wide_string_c); -#if 0 - // TODO(tsepez): ambiguos overload prevents compilation const wchar_t* c_string_same1 = L"hello"; ASSERT_TRUE(wide_string_c == c_string_same1); ASSERT_TRUE(c_string_same1 == wide_string_c); @@ -272,7 +259,6 @@ TEST(fxcrt, WideStringCOperatorEQ) { ASSERT_FALSE(c_string1 == wide_string_c); ASSERT_FALSE(c_string2 == wide_string_c); ASSERT_FALSE(c_string3 == wide_string_c); -#endif } TEST(fxcrt, WideStringCOperatorNE) { @@ -311,8 +297,6 @@ TEST(fxcrt, WideStringCOperatorNE) { ASSERT_TRUE(wide_string2 != wide_string_c); ASSERT_TRUE(wide_string3 != wide_string_c); -#if 0 - // See above TODO. const wchar_t* c_string_same1 = L"hello"; ASSERT_FALSE(wide_string_c != c_string_same1); ASSERT_FALSE(c_string_same1 != wide_string_c); @@ -327,5 +311,4 @@ TEST(fxcrt, WideStringCOperatorNE) { ASSERT_TRUE(c_string1 != wide_string_c); ASSERT_TRUE(c_string2 != wide_string_c); ASSERT_TRUE(c_string3 != wide_string_c); -#endif } |