Fix the code that causes warnings

xfa/fxbarcode/datamatrix/BC_ErrorCorrection.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
 // Original code is licensed as follows:
 /*
  * Copyright 2006 Jeremias Maerki.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ skipping 154 matching lines @@
 CFX_WideString CBC_ErrorCorrection::createECCBlock(CFX_WideString codewords,
                                                    int32_t numECWords,
                                                    int32_t& e) {
   return createECCBlock(codewords, 0, codewords.GetLength(), numECWords, e);
 }
 CFX_WideString CBC_ErrorCorrection::createECCBlock(CFX_WideString codewords,
                                                    int32_t start,
                                                    int32_t len,
                                                    int32_t numECWords,
                                                    int32_t& e) {
-  int32_t table = -1;
-  for (int32_t i = 0; i < sizeof(FACTOR_SETS) / sizeof(int32_t); i++) {
-    if (FACTOR_SETS[i] == numECWords) {
-      table = i;
-      break;
-    }
-  }
-  if (table < 0) {
+  static const size_t kFactorTableNum = sizeof(FACTOR_SETS) / sizeof(int32_t);
+  size_t table = 0;
+  while (table < kFactorTableNum && FACTOR_SETS[table] != numECWords)
+    table++;
+
+  if (table >= kFactorTableNum) {
     e = BCExceptionIllegalArgument;
     return (FX_WCHAR*)"";
   }
   uint16_t* ecc = FX_Alloc(uint16_t, numECWords);
   FXSYS_memset(ecc, 0, numECWords * sizeof(uint16_t));
   for (int32_t l = start; l < start + len; l++) {
     uint16_t m = ecc[numECWords - 1] ^ codewords.GetAt(l);
     for (int32_t k = numECWords - 1; k > 0; k--) {
       if (m != 0 && FACTORS[table][k] != 0) {
         ecc[k] = (uint16_t)(ecc[k - 1] ^
                             ALOG[(LOG[m] + LOG[FACTORS[table][k]]) % 255]);
       } else {
         ecc[k] = ecc[k - 1];
       }
     }
     if (m != 0 && FACTORS[table][0] != 0) {
       ecc[0] = (uint16_t)ALOG[(LOG[m] + LOG[FACTORS[table][0]]) % 255];
     } else {
       ecc[0] = 0;
     }
   }
   CFX_WideString strecc;
   for (int32_t j = 0; j < numECWords; j++) {
     strecc += (FX_WCHAR)ecc[numECWords - j - 1];
   }
   FX_Free(ecc);
   return strecc;
 }