Clean up more XFA code which causes warnings

This is part of efforts to bring XFA to chromium_code standard.
Most of them will have behavior change.
The details of these problems are:

xfa/fgas/layout/fgas_rtfbreak.cpp:
  Wrong condition with misused variable
xfa/fgas/localization/fgas_locale.cpp
  Unnecessary condition
xfa/fxbarcode/datamatrix/BC_DataMatrixDecodedBitStreamParser.cpp
xfa/fxfa/app/xfa_fffield.cpp
  Unreachable code should be the correct code
xfa/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp
xfa/fxbarcode/qrcode/BC_QRDetector.cpp
  Wrong condition logic

BUG=pdfium:29

Review-Url: https://codereview.chromium.org/2000073003

1 files changed, 2 insertions, 2 deletions

diff --git a/xfa/fxbarcode/datamatrix/BC_DataMatrixDecodedBitStreamParser.cpp b/xfa/fxbarcode/datamatrix/BC_DataMatrixDecodedBitStreamParser.cpp
index 53c5a84656..292a1de2f7 100644
--- a/xfa/fxbarcode/datamatrix/BC_DataMatrixDecodedBitStreamParser.cpp
+++ b/xfa/fxbarcode/datamatrix/BC_DataMatrixDecodedBitStreamParser.cpp
@@ -246,7 +246,7 @@ void CBC_DataMatrixDecodedBitStreamParser::DecodeC40Segment(
           shift = 0;
           break;
         default:
-          break;
+          // Should never be reached.
           e = BCExceptionFormatException;
           return;
       }
@@ -337,7 +337,7 @@ void CBC_DataMatrixDecodedBitStreamParser::DecodeTextSegment(
           }
           break;
         default:
-          break;
+          // Should never be reached.
           e = BCExceptionFormatException;
           return;
       }