From 3c3e271ea9fd19dd535a74b5546b27e3e82dcb81 Mon Sep 17 00:00:00 2001
From: Tom Sepez <tsepez@chromium.org>
Date: Mon, 17 Apr 2017 15:38:19 -0700
Subject: Use Byte/WideString iterators

Change-Id: I85c8423c177fd7ecd5da90ef89419efc0f9cf44b
Reviewed-on: https://pdfium-review.googlesource.com/4262
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
---
 fxbarcode/oned/BC_OnedCodaBarWriter.cpp | 15 ++++-----------
 fxbarcode/oned/BC_OnedCode128Writer.cpp | 16 ++++------------
 fxbarcode/oned/BC_OnedCode39Writer.cpp  |  6 +++---
 fxbarcode/oned/BC_OnedEAN13Writer.cpp   | 12 ++++--------
 4 files changed, 15 insertions(+), 34 deletions(-)

(limited to 'fxbarcode/oned')

diff --git a/fxbarcode/oned/BC_OnedCodaBarWriter.cpp b/fxbarcode/oned/BC_OnedCodaBarWriter.cpp
index 6c04cafe7b..d3706ab96d 100644
--- a/fxbarcode/oned/BC_OnedCodaBarWriter.cpp
+++ b/fxbarcode/oned/BC_OnedCodaBarWriter.cpp
@@ -108,18 +108,11 @@ bool CBC_OnedCodaBarWriter::FindChar(wchar_t ch, bool isContent) {
 }
 bool CBC_OnedCodaBarWriter::CheckContentValidity(
     const CFX_WideStringC& contents) {
-  wchar_t ch;
-  int32_t index = 0;
-  for (index = 0; index < contents.GetLength(); index++) {
-    ch = contents.GetAt(index);
-    if (FindChar(ch, false)) {
-      continue;
-    } else {
-      return false;
-    }
-  }
-  return true;
+  return std::all_of(
+      contents.begin(), contents.end(),
+      [this](const wchar_t& ch) { return this->FindChar(ch, false); });
 }
+
 CFX_WideString CBC_OnedCodaBarWriter::FilterContents(
     const CFX_WideStringC& contents) {
   CFX_WideString filtercontents;
diff --git a/fxbarcode/oned/BC_OnedCode128Writer.cpp b/fxbarcode/oned/BC_OnedCode128Writer.cpp
index 53f4d01449..e2b6823414 100644
--- a/fxbarcode/oned/BC_OnedCode128Writer.cpp
+++ b/fxbarcode/oned/BC_OnedCode128Writer.cpp
@@ -113,22 +113,14 @@ CFX_WideString CBC_OnedCode128Writer::FilterContents(
   }
   CFX_WideString filtercontents;
   if (m_codeFormat == BC_CODE128_B) {
-    for (int32_t i = 0; i < filterChineseChar.GetLength(); i++) {
-      ch = filterChineseChar.GetAt(i);
-      if (ch >= 32 && ch <= 126) {
+    for (const auto& ch : filterChineseChar) {
+      if (ch >= 32 && ch <= 126)
         filtercontents += ch;
-      } else {
-        continue;
-      }
     }
   } else if (m_codeFormat == BC_CODE128_C) {
-    for (int32_t i = 0; i < filterChineseChar.GetLength(); i++) {
-      ch = filterChineseChar.GetAt(i);
-      if (ch >= 32 && ch <= 106) {
+    for (const auto& ch : filterChineseChar) {
+      if (ch >= 32 && ch <= 106)
         filtercontents += ch;
-      } else {
-        continue;
-      }
     }
   } else {
     filtercontents = contents;
diff --git a/fxbarcode/oned/BC_OnedCode39Writer.cpp b/fxbarcode/oned/BC_OnedCode39Writer.cpp
index bcc64d83bd..b1a56cad40 100644
--- a/fxbarcode/oned/BC_OnedCode39Writer.cpp
+++ b/fxbarcode/oned/BC_OnedCode39Writer.cpp
@@ -163,11 +163,11 @@ char CBC_OnedCode39Writer::CalcCheckSum(const CFX_ByteString& contents,
   }
   int32_t checksum = 0;
   int32_t len = (int32_t)strlen(ALPHABET_STRING);
-  for (int32_t i = 0; i < contents.GetLength(); i++) {
+  for (const auto& c : contents) {
     int32_t j = 0;
     for (; j < len; j++) {
-      if (ALPHABET_STRING[j] == contents[i]) {
-        if (contents[i] != '*')
+      if (ALPHABET_STRING[j] == c) {
+        if (c != '*')
           checksum += j;
         break;
       }
diff --git a/fxbarcode/oned/BC_OnedEAN13Writer.cpp b/fxbarcode/oned/BC_OnedEAN13Writer.cpp
index a11938a4d9..96b51b0570 100644
--- a/fxbarcode/oned/BC_OnedEAN13Writer.cpp
+++ b/fxbarcode/oned/BC_OnedEAN13Writer.cpp
@@ -48,17 +48,13 @@ CBC_OnedEAN13Writer::CBC_OnedEAN13Writer() {
   m_codeWidth = 3 + (7 * 6) + 5 + (7 * 6) + 3;
 }
 CBC_OnedEAN13Writer::~CBC_OnedEAN13Writer() {}
+
 bool CBC_OnedEAN13Writer::CheckContentValidity(
     const CFX_WideStringC& contents) {
-  for (int32_t i = 0; i < contents.GetLength(); i++) {
-    if (contents.GetAt(i) >= '0' && contents.GetAt(i) <= '9') {
-      continue;
-    } else {
-      return false;
-    }
-  }
-  return true;
+  return std::all_of(contents.begin(), contents.end(),
+                     [](const wchar_t& w) { return w >= L'0' && w <= L'9'; });
 }
+
 CFX_WideString CBC_OnedEAN13Writer::FilterContents(
     const CFX_WideStringC& contents) {
   CFX_WideString filtercontents;
-- 
cgit v1.2.3