diff options
| author | tsepez <tsepez@chromium.org> | 2016-05-20 16:09:37 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-20 16:09:37 -0700 |
| commit | 76852be59cbfdb22901b37bb8579d19c07af7037 (patch) | |
| tree | 6c6c7798f62f089bc7d4720954ad2d07469582a5 /xfa/fxbarcode/pdf417 | |
| parent | 038aa531bcf1a76764d3cef46fd9b8e08b166dae (diff) | |
| download | pdfium-76852be59cbfdb22901b37bb8579d19c07af7037.tar.xz | |
Fix /analyze warnings in fxbarcode/
Part of the fix to the issues in bug 613623. These are all real bugs
arising from non-use of L""-style literals band-aided over with
casts. Remove the casts, and fix the literals or replace with empty
string constructors.
BUG=613623, 427616
Review-Url: https://codereview.chromium.org/1999933002
Diffstat (limited to 'xfa/fxbarcode/pdf417')
| -rw-r--r-- | xfa/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/xfa/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp b/xfa/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp index 27c6651542..7617d526c1 100644 --- a/xfa/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp +++ b/xfa/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp @@ -52,7 +52,9 @@ int32_t CBC_PDF417HighLevelEncoder::PUNCTUATION[128] = {0}; void CBC_PDF417HighLevelEncoder::Initialize() { Inverse(); } + void CBC_PDF417HighLevelEncoder::Finalize() {} + CFX_WideString CBC_PDF417HighLevelEncoder::encodeHighLevel( CFX_WideString wideMsg, Compaction compaction, @@ -65,7 +67,7 @@ CFX_WideString CBC_PDF417HighLevelEncoder::encodeHighLevel( FX_WCHAR ch = (FX_WCHAR)(bytes.GetAt(i) & 0xff); if (ch == '?' && bytes.GetAt(i) != '?') { e = BCExceptionCharactersOutsideISO88591Encoding; - return (FX_WCHAR*)""; + return CFX_WideString(); } msg += ch; } |