Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(163)

Unified Diff: third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp

Issue 2373983006: reflow comments in wtf/text (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp
diff --git a/third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp b/third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp
index 0ac4aead68b2c398119aad3e1174daa88338d660..92ec825b60ec493729eeabbc77a0ed367929e97c 100644
--- a/third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp
+++ b/third_party/WebKit/Source/wtf/text/TextCodecUTF16.cpp
@@ -150,8 +150,9 @@ CString TextCodecUTF16::encode(const UChar* characters,
char* bytes;
CString result = CString::newUninitialized(length * 2, bytes);
- // FIXME: CString is not a reasonable data structure for encoded UTF-16, which will have
- // null characters inside it. Perhaps the result of encode should not be a CString.
+ // FIXME: CString is not a reasonable data structure for encoded UTF-16, which
+ // will have null characters inside it. Perhaps the result of encode should
+ // not be a CString.
if (m_littleEndian) {
for (size_t i = 0; i < length; ++i) {
UChar c = characters[i];
@@ -172,7 +173,7 @@ CString TextCodecUTF16::encode(const UChar* characters,
CString TextCodecUTF16::encode(const LChar* characters,
size_t length,
UnencodableHandling) {
- // In the LChar case, we do actually need to perform this check in release. :)
dcheng 2016/10/01 03:06:14 Heh
+ // In the LChar case, we do actually need to perform this check in release. :)
RELEASE_ASSERT(length <= numeric_limits<size_t>::max() / 2);
char* bytes;

Powered by Google App Engine
This is Rietveld 408576698