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/widestring.h | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) (limited to 'core/fxcrt/widestring.h') 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. -- cgit v1.2.3