XFA: Fix a bunch of pointless returns.

xfa/src/fxfa/src/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_fm2js.h"
 #include <time.h>
 #define FINANCIAL_PRECISION 0.00000001
 struct XFA_FMHtmlReserveCode {
@@ skipping 2454 matching lines @@
   CFX_ByteString strIsoTime;
   strIsoTime.Format("%02d:%02d:%02d", iHour, iMin, iSec);
   if (bGM) {
     iRet = GetGMTTime(hThis, strIsoTime, szFormat, szLocale, strTime);
   } else {
     iRet = IsoTime2Local(hThis, strIsoTime, szFormat, szLocale, strTime);
   }
   if (!iRet) {
     strTime = "";
   }
-  return;
 }
+
 void CXFA_FM2JSContext::GetLocalTimeZone(int32_t& iHour,
                                          int32_t& iMin,
                                          int32_t& iSec) {
   time_t now;
   time(&now);
   struct tm* pGmt = gmtime(&now);
   int32_t iGMHour = pGmt->tm_hour;
   int32_t iGMMin = pGmt->tm_min;
   int32_t iGMSec = pGmt->tm_sec;
   struct tm* pLocal = localtime(&now);
@@ skipping 4706 matching lines @@
   FXSYS_assert(pAppProvider);
   CFX_WideString wsFormat;
   pAppProvider->LoadString(iStringID, wsFormat);
   CFX_WideString wsMessage;
   va_list arg_ptr;
   va_start(arg_ptr, iStringID);
   wsMessage.FormatV((const FX_WCHAR*)wsFormat, arg_ptr);
   va_end(arg_ptr);
   FXJSE_ThrowMessage("", FX_UTF8Encode(wsMessage, wsMessage.GetLength()));
 }