Revert "Revert "Cleanup some numeric code.""

fpdfsdk/src/fsdk_baseannot.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../include/fsdk_define.h"
 #include "../include/fsdk_mgr.h"
 #include "../include/fsdk_baseannot.h"
 
@@ skipping 203 matching lines @@
 
 CPDFSDK_DateTime& CPDFSDK_DateTime::FromPDFDateTimeString(
     const CFX_ByteString& dtStr) {
   int strLength = dtStr.GetLength();
   if (strLength > 0) {
     int i = 0;
     int j, k;
     FX_CHAR ch;
     while (i < strLength) {
       ch = dtStr[i];
-      if (ch >= '0' && ch <= '9')
+      if (std::isdigit(ch))
         break;
       i++;
     }
     if (i >= strLength)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 4) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.year = (int16_t)k;
     if (i >= strLength || j < 4)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.month = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.day = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.hour = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.minute = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
 
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.second = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
 
     ch = dtStr[i++];
     if (ch != '-' && ch != '+')
       return *this;
     if (ch == '-')
       dt.tzHour = -1;
     else
       dt.tzHour = 1;
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.tzHour *= (FX_CHAR)k;
     if (i >= strLength || j < 2)
       return *this;
 
     ch = dtStr[i++];
     if (ch != '\'')
       return *this;
     j = 0;
     k = 0;
     while (i < strLength && j < 2) {
       ch = dtStr[i];
       k = k * 10 + ch - '0';
       j++;
-      if (ch < '0' || ch > '9')
+      if (!std::isdigit(ch))
         break;
       i++;
     }
     dt.tzMinute = (uint8_t)k;
     if (i >= strLength || j < 2)
       return *this;
   }
 
   return *this;
 }
@@ skipping 641 matching lines @@
                            CPDF_Annot::Normal, NULL);
 
   return;
 }
 
 CPDF_Page* CPDFSDK_Annot::GetPDFPage() {
   if (m_pPageView)
     return m_pPageView->GetPDFPage();
   return NULL;
 }