From ac7ce6ec217ba7b4753f4cf2ed80c921d8ea6898 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 6 Nov 2017 18:51:02 +0000 Subject: Mark string pointers as const in string unit tests. Change-Id: I4895ee9c0cbe1aaf1fe9913d39ce11e6ad855dd3 Reviewed-on: https://pdfium-review.googlesource.com/17873 Reviewed-by: Tom Sepez Commit-Queue: Lei Zhang --- core/fxcrt/widestring_unittest.cpp | 40 +++++++++++++++++++------------------- 1 file changed, 20 insertions(+), 20 deletions(-) (limited to 'core/fxcrt/widestring_unittest.cpp') diff --git a/core/fxcrt/widestring_unittest.cpp b/core/fxcrt/widestring_unittest.cpp index 7824c2bd72..681fb472a2 100644 --- a/core/fxcrt/widestring_unittest.cpp +++ b/core/fxcrt/widestring_unittest.cpp @@ -152,8 +152,8 @@ TEST(WideString, OperatorEQ) { EXPECT_FALSE(wide_string_c2 == wide_string); EXPECT_FALSE(wide_string_c3 == wide_string); - const wchar_t* c_null_string = nullptr; - const wchar_t* c_empty_string = L""; + const wchar_t* const c_null_string = nullptr; + const wchar_t* const c_empty_string = L""; EXPECT_TRUE(null_string == c_null_string); EXPECT_TRUE(null_string == c_empty_string); EXPECT_TRUE(empty_string == c_null_string); @@ -167,13 +167,13 @@ TEST(WideString, OperatorEQ) { EXPECT_TRUE(c_null_string == deleted_string); EXPECT_TRUE(c_empty_string == deleted_string); - const wchar_t* c_string_same1 = L"hello"; + const wchar_t* const c_string_same1 = L"hello"; EXPECT_TRUE(wide_string == c_string_same1); EXPECT_TRUE(c_string_same1 == wide_string); - const wchar_t* c_string1 = L"he"; - const wchar_t* c_string2 = L"hellp"; - const wchar_t* c_string3 = L"hellod"; + const wchar_t* const c_string1 = L"he"; + const wchar_t* const c_string2 = L"hellp"; + const wchar_t* const c_string3 = L"hellod"; EXPECT_FALSE(wide_string == c_string1); EXPECT_FALSE(wide_string == c_string2); EXPECT_FALSE(wide_string == c_string3); @@ -253,8 +253,8 @@ TEST(WideString, OperatorNE) { EXPECT_TRUE(wide_string_c2 != wide_string); EXPECT_TRUE(wide_string_c3 != wide_string); - const wchar_t* c_null_string = nullptr; - const wchar_t* c_empty_string = L""; + const wchar_t* const c_null_string = nullptr; + const wchar_t* const c_empty_string = L""; EXPECT_FALSE(null_string != c_null_string); EXPECT_FALSE(null_string != c_empty_string); EXPECT_FALSE(empty_string != c_null_string); @@ -268,13 +268,13 @@ TEST(WideString, OperatorNE) { EXPECT_FALSE(c_null_string != deleted_string); EXPECT_FALSE(c_empty_string != deleted_string); - const wchar_t* c_string_same1 = L"hello"; + const wchar_t* const c_string_same1 = L"hello"; EXPECT_FALSE(wide_string != c_string_same1); EXPECT_FALSE(c_string_same1 != wide_string); - const wchar_t* c_string1 = L"he"; - const wchar_t* c_string2 = L"hellp"; - const wchar_t* c_string3 = L"hellod"; + const wchar_t* const c_string1 = L"he"; + const wchar_t* const c_string2 = L"hellp"; + const wchar_t* const c_string3 = L"hellod"; EXPECT_TRUE(wide_string != c_string1); EXPECT_TRUE(wide_string != c_string2); EXPECT_TRUE(wide_string != c_string3); @@ -940,13 +940,13 @@ TEST(WideStringView, OperatorEQ) { EXPECT_FALSE(wide_string2 == wide_string_c); EXPECT_FALSE(wide_string3 == wide_string_c); - const wchar_t* c_string_same1 = L"hello"; + const wchar_t* const c_string_same1 = L"hello"; EXPECT_TRUE(wide_string_c == c_string_same1); EXPECT_TRUE(c_string_same1 == wide_string_c); - const wchar_t* c_string1 = L"he"; - const wchar_t* c_string2 = L"hellp"; - const wchar_t* c_string3 = L"hellod"; + const wchar_t* const c_string1 = L"he"; + const wchar_t* const c_string2 = L"hellp"; + const wchar_t* const c_string3 = L"hellod"; EXPECT_FALSE(wide_string_c == c_string1); EXPECT_FALSE(wide_string_c == c_string2); EXPECT_FALSE(wide_string_c == c_string3); @@ -992,13 +992,13 @@ TEST(WideStringView, OperatorNE) { EXPECT_TRUE(wide_string2 != wide_string_c); EXPECT_TRUE(wide_string3 != wide_string_c); - const wchar_t* c_string_same1 = L"hello"; + const wchar_t* const c_string_same1 = L"hello"; EXPECT_FALSE(wide_string_c != c_string_same1); EXPECT_FALSE(c_string_same1 != wide_string_c); - const wchar_t* c_string1 = L"he"; - const wchar_t* c_string2 = L"hellp"; - const wchar_t* c_string3 = L"hellod"; + const wchar_t* const c_string1 = L"he"; + const wchar_t* const c_string2 = L"hellp"; + const wchar_t* const c_string3 = L"hellod"; EXPECT_TRUE(wide_string_c != c_string1); EXPECT_TRUE(wide_string_c != c_string2); EXPECT_TRUE(wide_string_c != c_string3); -- cgit v1.2.3