Fix all the code which has duplicate variable declarations

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 629 matching lines @@
         return;
       }
 
       std::unique_ptr<CFXJSE_Value> propertyValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> jsObjectValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> newPropertyValue(
           new CFXJSE_Value(pIsolate));
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 1, propertyValue.get());
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 2, jsObjectValue.get());
       if (FXJSE_Value_IsNull(propertyValue.get())) {
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           GetObjectDefaultValue(jsObjectValue.get(), newPropertyValue.get());
           if (!FXJSE_Value_IsNull(newPropertyValue.get()))
             iCount++;
         }
       } else {
         CFX_ByteString propertyStr;
         FXJSE_Value_ToUTF8String(propertyValue.get(), propertyStr);
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           FXJSE_Value_GetObjectProp(jsObjectValue.get(),
                                     propertyStr.AsStringC(),
                                     newPropertyValue.get());
           iCount += (FXJSE_Value_IsNull(newPropertyValue.get()) ? 0 : 1);
         }
       }
     } else if (FXJSE_Value_IsObject(argValue.get())) {
       std::unique_ptr<CFXJSE_Value> newPropertyValue(
           new CFXJSE_Value(pIsolate));
@@ skipping 51 matching lines @@
         return;
       }
 
       std::unique_ptr<CFXJSE_Value> propertyValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> jsObjectValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> newPropertyValue(
           new CFXJSE_Value(pIsolate));
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 1, propertyValue.get());
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 2, jsObjectValue.get());
       if (FXJSE_Value_IsNull(propertyValue.get())) {
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           GetObjectDefaultValue(jsObjectValue.get(), newPropertyValue.get());
           if (FXJSE_Value_IsNull(newPropertyValue.get()))
             continue;
 
           uCount++;
           FX_DOUBLE dValue = ValueToDouble(pThis, newPropertyValue.get());
           dMaxValue = (uCount == 1) ? dValue : std::max(dMaxValue, dValue);
         }
       } else {
         CFX_ByteString propertyStr;
         FXJSE_Value_ToUTF8String(propertyValue.get(), propertyStr);
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           FXJSE_Value_GetObjectProp(jsObjectValue.get(),
                                     propertyStr.AsStringC(),
                                     newPropertyValue.get());
           if (FXJSE_Value_IsNull(newPropertyValue.get()))
             continue;
 
           uCount++;
           FX_DOUBLE dValue = ValueToDouble(pThis, newPropertyValue.get());
           dMaxValue = (uCount == 1) ? dValue : std::max(dMaxValue, dValue);
@@ skipping 46 matching lines @@
         return;
       }
 
       std::unique_ptr<CFXJSE_Value> propertyValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> jsObjectValue(new CFXJSE_Value(pIsolate));
       std::unique_ptr<CFXJSE_Value> newPropertyValue(
           new CFXJSE_Value(pIsolate));
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 1, propertyValue.get());
       FXJSE_Value_GetObjectPropByIdx(argValue.get(), 2, jsObjectValue.get());
       if (FXJSE_Value_IsNull(propertyValue.get())) {
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           GetObjectDefaultValue(jsObjectValue.get(), newPropertyValue.get());
           if (FXJSE_Value_IsNull(newPropertyValue.get()))
             continue;
 
           uCount++;
           FX_DOUBLE dValue = ValueToDouble(pThis, newPropertyValue.get());
           dMinValue = uCount == 1 ? dValue : std::min(dMinValue, dValue);
         }
       } else {
         CFX_ByteString propertyStr;
         FXJSE_Value_ToUTF8String(propertyValue.get(), propertyStr);
-        for (int32_t i = 2; i < iLength; i++) {
-          FXJSE_Value_GetObjectPropByIdx(argValue.get(), i,
+        for (int32_t j = 2; j < iLength; j++) {
+          FXJSE_Value_GetObjectPropByIdx(argValue.get(), j,
                                          jsObjectValue.get());
           FXJSE_Value_GetObjectProp(jsObjectValue.get(),
                                     propertyStr.AsStringC(),
                                     newPropertyValue.get());
           if (FXJSE_Value_IsNull(newPropertyValue.get()))
             continue;
 
           uCount++;
           FX_DOUBLE dValue = ValueToDouble(pThis, newPropertyValue.get());
           dMinValue = uCount == 1 ? dValue : std::min(dMinValue, dValue);
@@ skipping 1095 matching lines @@
         ++iPos;
       } else if (iPos == 2) {
         iSecond = FXSYS_atoi(strTemp);
         ++iPos;
       }
       iIndex += 2;
     }
   }
   if (*(pData + iIndex) == '.') {
     ++iIndex;
-    FX_CHAR strTemp[4];
-    strTemp[3] = '\0';
+    FX_CHAR strSec[4];
+    strSec[3] = '\0';
     if (*(pData + iIndex) > '9' || *(pData + iIndex) < '0') {
       return iRet;
     }
-    strTemp[0] = *(pData + iIndex);
+    strSec[0] = *(pData + iIndex);
     if (*(pData + iIndex + 1) > '9' || *(pData + iIndex + 1) < '0') {
       return iRet;
     }
-    strTemp[1] = *(pData + iIndex + 1);
+    strSec[1] = *(pData + iIndex + 1);
     if (*(pData + iIndex + 2) > '9' || *(pData + iIndex + 2) < '0') {
       return iRet;
     }
-    strTemp[2] = *(pData + iIndex + 2);
-    iMilliSecond = FXSYS_atoi(strTemp);
+    strSec[2] = *(pData + iIndex + 2);
+    iMilliSecond = FXSYS_atoi(strSec);
     if (iMilliSecond > 100) {
       iMilliSecond = 0;
       return iRet;
     }
     iIndex += 3;
   }
   int32_t iSign = 1;
   if (*(pData + iIndex) == 'z' || *(pData + iIndex) == 'Z') {
     iRet = 1;
     return iRet;
@@ skipping 1055 matching lines @@
     pContext->ThrowException(XFA_IDS_INCORRECT_NUMBER_OF_METHOD, L"Term");
   }
 }
 
 // static
 void CXFA_FM2JSContext::Choose(CFXJSE_Value* pThis,
                                const CFX_ByteStringC& szFuncName,
                                CFXJSE_Arguments& args) {
   CXFA_FM2JSContext* pContext =
       static_cast<CXFA_FM2JSContext*>(FXJSE_Value_ToObject(pThis, nullptr));
-  v8::Isolate* pIsolate = pContext->GetScriptRuntime();
   int32_t argc = args.GetLength();
   if (argc > 1) {
+    v8::Isolate* pIsolate = pContext->GetScriptRuntime();
     std::unique_ptr<CFXJSE_Value> argOne = args.GetValue(0);
     FX_BOOL argOneIsNull = FALSE;
     int32_t iIndex = 0;
     argOneIsNull = ValueIsNull(pThis, argOne.get());
     if (!argOneIsNull) {
       iIndex = (int32_t)ValueToFloat(pThis, argOne.get());
     }
     if (argOneIsNull) {
       FXJSE_Value_SetNull(args.GetReturnValue());
     } else if (iIndex < 1) {
@@ skipping 189 matching lines @@
     pContext->ThrowException(XFA_IDS_INCORRECT_NUMBER_OF_METHOD, L"If");
   }
 }
 
 // static
 void CXFA_FM2JSContext::Eval(CFXJSE_Value* pThis,
                              const CFX_ByteStringC& szFuncName,
                              CFXJSE_Arguments& args) {
   CXFA_FM2JSContext* pContext =
       static_cast<CXFA_FM2JSContext*>(FXJSE_Value_ToObject(pThis, nullptr));
-  v8::Isolate* pIsolate = pContext->GetScriptRuntime();
   if (args.GetLength() == 1) {
+    v8::Isolate* pIsolate = pContext->GetScriptRuntime();
     std::unique_ptr<CFXJSE_Value> scriptValue = GetSimpleValue(pThis, args, 0);
     CFX_ByteString utf8ScriptString;
     ValueToUTF8String(scriptValue.get(), utf8ScriptString);
     if (utf8ScriptString.IsEmpty()) {
       FXJSE_Value_SetNull(args.GetReturnValue());
     } else {
       CFX_WideTextBuf wsJavaScriptBuf;
       CFX_WideString javaScript;
       CFX_WideString wsError;
       CXFA_FM2JSContext::Translate(
           CFX_WideString::FromUTF8(utf8ScriptString.AsStringC()).AsStringC(),
           wsJavaScriptBuf, wsError);
-      CFXJSE_Context* pContext =
+      CFXJSE_Context* pNewContext =
           FXJSE_Context_Create(pIsolate, nullptr, nullptr);
       std::unique_ptr<CFXJSE_Value> returnValue(new CFXJSE_Value(pIsolate));
       javaScript = wsJavaScriptBuf.AsStringC();
       FXJSE_ExecuteScript(
-          pContext,
+          pNewContext,
           FX_UTF8Encode(javaScript.c_str(), javaScript.GetLength()).c_str(),
           returnValue.get());
       FXJSE_Value_Set(args.GetReturnValue(), returnValue.get());
-      FXJSE_Context_Release(pContext);
+      FXJSE_Context_Release(pNewContext);
     }
   } else {
     pContext->ThrowException(XFA_IDS_INCORRECT_NUMBER_OF_METHOD, L"Eval");
   }
 }
 
 // static
 void CXFA_FM2JSContext::Ref(CFXJSE_Value* pThis,
                             const CFX_ByteStringC& szFuncName,
                             CFXJSE_Arguments& args) {
@@ skipping 228 matching lines @@
           }
           strFirstUnit += (*(pData + u));
           ++u;
         }
         strFirstUnit.MakeLower();
         if (argc == 2) {
           std::unique_ptr<CFXJSE_Value> unitValue =
               GetSimpleValue(pThis, args, 1);
           CFX_ByteString unitTempString;
           ValueToUTF8String(unitValue.get(), unitTempString);
-          const FX_CHAR* pData = unitTempString.c_str();
-          int32_t u = 0;
-          while (*(pData + u) == ' ' || *(pData + u) == 0x09 ||
-                 *(pData + u) == 0x0B || *(pData + u) == 0x0C ||
-                 *(pData + u) == 0x0A || *(pData + u) == 0x0D) {
-            ++u;
+          const FX_CHAR* pChar = unitTempString.c_str();
+          int32_t uVal = 0;
+          while (*(pChar + uVal) == ' ' || *(pChar + uVal) == 0x09 ||
+                 *(pChar + uVal) == 0x0B || *(pChar + uVal) == 0x0C ||
+                 *(pChar + uVal) == 0x0A || *(pChar + uVal) == 0x0D) {
+            ++uVal;
           }
-          while (u < unitTempString.GetLength()) {
-            if ((*(pData + u) > '9' || *(pData + u) < '0') &&
-                *(pData + u) != '.') {
+          while (uVal < unitTempString.GetLength()) {
+            if ((*(pChar + uVal) > '9' || *(pChar + uVal) < '0') &&
+                *(pChar + uVal) != '.') {
               break;
             }
-            ++u;
+            ++uVal;
           }
-          while (*(pData + u) == ' ' || *(pData + u) == 0x09 ||
-                 *(pData + u) == 0x0B || *(pData + u) == 0x0C ||
-                 *(pData + u) == 0x0A || *(pData + u) == 0x0D) {
-            ++u;
+          while (*(pChar + uVal) == ' ' || *(pChar + uVal) == 0x09 ||
+                 *(pChar + uVal) == 0x0B || *(pChar + uVal) == 0x0C ||
+                 *(pChar + uVal) == 0x0A || *(pChar + uVal) == 0x0D) {
+            ++uVal;
           }
-          int32_t uLen = unitTempString.GetLength();
-          while (u < uLen) {
-            if (*(pData + u) == ' ') {
+          int32_t uValLen = unitTempString.GetLength();
+          while (uVal < uValLen) {
+            if (*(pChar + uVal) == ' ') {
               break;
             }
-            strUnit += (*(pData + u));
-            ++u;
+            strUnit += (*(pChar + uVal));
+            ++uVal;
           }
           strUnit.MakeLower();
         } else {
           strUnit = strFirstUnit;
         }
         FX_DOUBLE dResult = 0;
         if (strFirstUnit == "in" || strFirstUnit == "inches") {
           if (strUnit == "mm" || strUnit == "millimeters") {
             dResult = dFirstNumber * 25.4;
           } else if (strUnit == "cm" || strUnit == "centimeters") {
@@ skipping 1051 matching lines @@
             wsTestPattern = FX_WSTRC(L"num{") + wsPattern;
             wsTestPattern += FX_WSTRC(L"}");
             CXFA_LocaleValue localeValue(XFA_VT_FLOAT, wsValue, wsTestPattern,
                                          pLocale, (CXFA_LocaleMgr*)pMgr);
             if (localeValue.IsValid()) {
               FXJSE_Value_SetDouble(args.GetReturnValue(),
                                     localeValue.GetDoubleNum());
             } else {
               wsTestPattern = FX_WSTRC(L"text{") + wsPattern;
               wsTestPattern += FX_WSTRC(L"}");
-              CXFA_LocaleValue localeValue(XFA_VT_TEXT, wsValue, wsTestPattern,
-                                           pLocale, (CXFA_LocaleMgr*)pMgr);
-              if (localeValue.IsValid()) {
-                szParsedValue = FX_UTF8Encode(localeValue.GetValue());
+              CXFA_LocaleValue localeValue2(XFA_VT_TEXT, wsValue, wsTestPattern,
+                                            pLocale, (CXFA_LocaleMgr*)pMgr);
+              if (localeValue2.IsValid()) {
+                szParsedValue = FX_UTF8Encode(localeValue2.GetValue());
                 FXJSE_Value_SetUTF8String(args.GetReturnValue(),
                                           szParsedValue.AsStringC());
               } else {
                 FXJSE_Value_SetUTF8String(args.GetReturnValue(), "");
               }
             }
           } break;
         }
       }
     }
@@ skipping 658 matching lines @@
         TrillionUS(CFX_ByteStringC(pData + iIndex, iCount), strBuf);
         iIndex += iCount;
         if (iIndex < iInteger) {
           strBuf << " Trillion ";
         }
       }
       strBuf << " Dollars";
       if (iInteger < iLength) {
         strBuf << " And ";
         iIndex = iInteger + 1;
-        int32_t iCount = 0;
         while (iIndex < iLength) {
-          iCount = (iLength - iIndex) % 12;
-          if (!iCount && iLength - iIndex > 0) {
-            iCount = 12;
-          }
-          TrillionUS(CFX_ByteStringC(pData + iIndex, iCount), strBuf);
-          iIndex += iCount;
+          int32_t iSize = (iLength - iIndex) % 12;
+          if (!iSize && iLength - iIndex > 0)
+            iSize = 12;
+          TrillionUS(CFX_ByteStringC(pData + iIndex, iSize), strBuf);
+          iIndex += iSize;
           if (iIndex < iLength) {
             strBuf << " Trillion ";
           }
         }
         strBuf << " Cents";
       }
     } break;
     default:
       break;
   }
@@ skipping 1716 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());
 }