Fix some signed/unsigned comparisons in xfa_fm2jscontext.cpp

xfa/fxfa/fm2js/xfa_fm2jscontext.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 "xfa/fxfa/fm2js/xfa_fm2jscontext.h"
 
 #include <time.h>
 
@@ skipping 340 matching lines @@
     formcalc_fm2js_functions,                // methods
     0,                                       // number of properties
     FX_ArraySize(formcalc_fm2js_functions),  // number of methods
     nullptr,                                 // dynamic prop type
     nullptr,                                 // dynamic prop getter
     nullptr,                                 // dynamic prop setter
     nullptr,                                 // dynamic prop deleter
     nullptr,                                 // dynamic prop method call
 };
 
 const uint8_t g_sAltTable_Date[] = {

[Lei Zhang, 2016/06/01 18:01:35]

Maybe static_assert(FX_ArraySize(g_sAltTable_Date) == L'a' - L'A" + 1) ? <- hope
I did that right.

[Oliver Chang, 2016/06/01 19:09:51]

Done.

-    255, 255, 255, 3,   9,   255, 255, 255, 255, 255, 255, 255, 2,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 1,   255,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
+    255, 255, 255, 3,   9,   255, 255, 255, 255, 255, 255,
+    255, 2,   255, 255, 255, 255, 255, 255, 255, 255, 255,
+    255, 255, 1,   255, 255, 255, 255, 255, 255, 255, 255,
 };
 
 const uint8_t g_sAltTable_Time[] = {
-    14,  255, 255, 3,   9,   255, 255, 15,  255, 255, 255, 255, 6,
-    255, 255, 255, 255, 255, 7,   255, 255, 255, 255, 255, 1,   17,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
-    15,  255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
-    255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
+    14,  255, 255, 3,   9,   255, 255, 15,  255, 255, 255,
+    255, 6,   255, 255, 255, 255, 255, 7,   255, 255, 255,
+    255, 255, 1,   17,  255, 255, 255, 255, 255, 255, 255,
 };
 
 void AlternateDateTimeSymbols(CFX_WideString& wsPattern,
                               const CFX_WideString& wsAltSymbols,
                               const uint8_t* pAltTable) {
   int32_t nLength = wsPattern.GetLength();
   FX_BOOL bInConstRange = FALSE;
   FX_BOOL bEscape = FALSE;
-  int32_t i = 0, n = 0;
+  int32_t i = 0;
   while (i < nLength) {
     FX_WCHAR wc = wsPattern[i];
     if (wc == L'\'') {
       bInConstRange = !bInConstRange;
       if (bEscape) {
         i++;
       } else {
         wsPattern.Delete(i);
         nLength--;
       }
       bEscape = !bEscape;
       continue;
     }
-    if (!bInConstRange && (n = wc - L'A') >= 0 && n <= (L'a' - L'A')) {
-      int32_t nAlt = (int32_t)pAltTable[n];
+    if (!bInConstRange && wc >= L'A' && wc <= L'a') {
+      uint8_t nAlt = pAltTable[wc - L'A'];
       if (nAlt != 255)
         wsPattern.SetAt(i, wsAltSymbols[nAlt]);
     }
     i++;
     bEscape = FALSE;
   }
 }
 
 bool PatternStringType(const CFX_ByteStringC& szPattern,
                        uint32_t& patternType) {
@@ skipping 3812 matching lines @@
       strEncode[2] = strCode[ch - iIndex * 16];
       wsResultBuf << FX_WSTRC(strEncode);
     } else if (ch <= 0x1f || ch == 0x7f) {
       int32_t iIndex = ch / 16;
       strEncode[1] = strCode[iIndex];
       strEncode[2] = strCode[ch - iIndex * 16];
       wsResultBuf << FX_WSTRC(strEncode);
     } else if (ch >= 0x20 && ch <= 0x7e) {
       wsResultBuf.AppendChar(ch);
     } else {
-      int32_t iRadix = 16;
+      const FX_WCHAR iRadix = 16;
       CFX_WideString strTmp;
       while (ch >= iRadix) {
         FX_WCHAR tmp = strCode[ch % iRadix];
         ch /= iRadix;
         strTmp += tmp;
       }
       strTmp += strCode[ch];
       int32_t iLen = strTmp.GetLength();
       if (iLen < 2) {
         break;
@@ skipping 2903 matching lines @@
   CFX_WideString wsFormat;
   pAppProvider->LoadString(iStringID, wsFormat);
   CFX_WideString wsMessage;
   va_list arg_ptr;
   va_start(arg_ptr, iStringID);
   wsMessage.FormatV(wsFormat.c_str(), arg_ptr);
   va_end(arg_ptr);
   FXJSE_ThrowMessage(
       "", FX_UTF8Encode(wsMessage.c_str(), wsMessage.GetLength()).AsStringC());
 }