commit 324026f1bf1b3598cb3c71ad96d13f86921c2c3c
author Tom Sepez <tsepez@chromium.org> Fri Mar 22 23:33:34 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Mar 22 23:33:34 2019 +0000
tree 9b988a190261a183b8c6c5d71d233d051586ac41
parent ad046fa88e1b5abbc65c0acab632bb39bc95eb21

Get rid of some needless pointer arithmetic in fx_string.cpp.

Change-Id: I57b00ac7abc7cdaeb9a9a6573c48e6061a27a027
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/51992
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

core/fxcrt/fx_string.cpp

diff --git a/core/fxcrt/fx_string.cpp b/core/fxcrt/fx_string.cpp
index b55931c..cb7465e 100644
--- a/core/fxcrt/fx_string.cpp
+++ b/core/fxcrt/fx_string.cpp
@@ -15,11 +15,9 @@
 #include "third_party/base/compiler_specific.h"
 
 ByteString FX_UTF8Encode(WideStringView wsStr) {
-  size_t len = wsStr.GetLength();
-  const wchar_t* pStr = wsStr.unterminated_c_str();
   CFX_UTF8Encoder encoder;
-  while (len-- > 0)
-    encoder.Input(*pStr++);
+  for (size_t i = 0; i < wsStr.GetLength(); ++i)
+    encoder.Input(wsStr[i]);
 
   return ByteString(encoder.GetResult());
 }