From ce6979f8d064507300fe0b3a856687fb958c9a5c Mon Sep 17 00:00:00 2001 From: Henrique Nakashima Date: Thu, 7 Dec 2017 22:19:53 +0000 Subject: Add WARN_UNUSED_RESULT to more static methods in (Byte|Wide)String. Also fixes some usages that were not working as intended after the conversion to static. Change-Id: I18c18369754f6ca165f98999b5b80eecf6c76973 Reviewed-on: https://pdfium-review.googlesource.com/20590 Reviewed-by: Lei Zhang Commit-Queue: Henrique Nakashima --- core/fxcrt/bytestring.h | 2 +- core/fxcrt/widestring.h | 17 ++++++++++------- core/fxcrt/widestring_unittest.cpp | 2 +- fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp | 9 +++------ 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/core/fxcrt/bytestring.h b/core/fxcrt/bytestring.h index 6b167802d5..916820656d 100644 --- a/core/fxcrt/bytestring.h +++ b/core/fxcrt/bytestring.h @@ -64,7 +64,7 @@ class ByteString { void clear() { m_pData.Reset(); } - static ByteString FromUnicode(const WideString& str); + static ByteString FromUnicode(const WideString& str) WARN_UNUSED_RESULT; // Explicit conversion to C-style string. // Note: Any subsequent modification of |this| will invalidate the result. diff --git a/core/fxcrt/widestring.h b/core/fxcrt/widestring.h index 14621ec1a1..728ab35090 100644 --- a/core/fxcrt/widestring.h +++ b/core/fxcrt/widestring.h @@ -33,8 +33,9 @@ class WideString { using const_iterator = const CharType*; using const_reverse_iterator = std::reverse_iterator; - static WideString Format(const wchar_t* lpszFormat, ...); - static WideString FormatV(const wchar_t* lpszFormat, va_list argList); + static WideString Format(const wchar_t* lpszFormat, ...) WARN_UNUSED_RESULT; + static WideString FormatV(const wchar_t* lpszFormat, + va_list argList) WARN_UNUSED_RESULT; WideString(); WideString(const WideString& other); @@ -58,13 +59,15 @@ class WideString { ~WideString(); - static WideString FromLocal(const ByteStringView& str); - static WideString FromCodePage(const ByteStringView& str, uint16_t codepage); + static WideString FromLocal(const ByteStringView& str) WARN_UNUSED_RESULT; + static WideString FromCodePage(const ByteStringView& str, + uint16_t codepage) WARN_UNUSED_RESULT; - static WideString FromUTF8(const ByteStringView& str); - static WideString FromUTF16LE(const unsigned short* str, size_t len); + static WideString FromUTF8(const ByteStringView& str) WARN_UNUSED_RESULT; + static WideString FromUTF16LE(const unsigned short* str, + size_t len) WARN_UNUSED_RESULT; - static size_t WStringLength(const unsigned short* str); + static size_t WStringLength(const unsigned short* str) WARN_UNUSED_RESULT; // Explicit conversion to C-style wide string. // Note: Any subsequent modification of |this| will invalidate the result. diff --git a/core/fxcrt/widestring_unittest.cpp b/core/fxcrt/widestring_unittest.cpp index 39337aa7a7..1673c8f069 100644 --- a/core/fxcrt/widestring_unittest.cpp +++ b/core/fxcrt/widestring_unittest.cpp @@ -1330,7 +1330,7 @@ TEST(WideString, FormatPrecision) { } TEST(WideString, FormatOutOfRangeChar) { - WideString::Format(L"unsupported char '%c'", 0x00FF00FF); + EXPECT_NE(L"", WideString::Format(L"unsupported char '%c'", 0x00FF00FF)); } TEST(WideString, Empty) { diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp index 90431243a4..70b0f1b514 100644 --- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp +++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp @@ -642,8 +642,7 @@ bool CPDFXFA_DocEnvironment::OnBeforeNotifySubmit() { if (!pFormFillEnv) return false; - WideString ws; - ws.FromLocal(IDS_XFA_Validate_Input); + WideString ws = WideString::FromLocal(IDS_XFA_Validate_Input); ByteString bs = ws.UTF16LE_Encode(); int len = bs.GetLength(); pFormFillEnv->Alert((FPDF_WIDESTRING)bs.GetBuffer(len), @@ -902,8 +901,7 @@ bool CPDFXFA_DocEnvironment::SubmitDataInternal(CXFA_FFDoc* hDoc, WideString csURL = submitData.GetSubmitTarget(); if (csURL.IsEmpty()) { - WideString ws; - ws.FromLocal("Submit cancelled."); + WideString ws = WideString::FromLocal("Submit cancelled."); ByteString bs = ws.UTF16LE_Encode(); int len = bs.GetLength(); pFormFillEnv->Alert(reinterpret_cast(bs.GetBuffer(len)), @@ -919,8 +917,7 @@ bool CPDFXFA_DocEnvironment::SubmitDataInternal(CXFA_FFDoc* hDoc, WideString csContent = submitData.GetSubmitXDPContent(); csContent.Trim(); - WideString space; - space.FromLocal(" "); + WideString space = WideString::FromLocal(" "); csContent = space + csContent + space; FPDF_DWORD flag = 0; if (submitData.IsSubmitEmbedPDF()) -- cgit v1.2.3