diff options
| author | weili <weili@chromium.org> | 2016-05-16 13:53:42 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-16 13:53:42 -0700 |
| commit | 3cc01f2ba255f4b7584668ee2b8e5ed97792c26d (patch) | |
| tree | 6a207d8910696ef1c28ef60a855ae266f81067cd /xfa/fgas/localization/fgas_locale.cpp | |
| parent | c6450bb06b69528406a2a261c70c4ea769965a8d (diff) | |
| download | pdfium-3cc01f2ba255f4b7584668ee2b8e5ed97792c26d.tar.xz | |
Fix the code that causes warnings
These are the left or newly added code which causes compilation
warnings of "signed and unsigned comparison". Need to fix them
before I re-enable the warning flag.
BUG=pdfium:29
Review-Url: https://codereview.chromium.org/1986533002
Diffstat (limited to 'xfa/fgas/localization/fgas_locale.cpp')
| -rw-r--r-- | xfa/fgas/localization/fgas_locale.cpp | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/xfa/fgas/localization/fgas_locale.cpp b/xfa/fgas/localization/fgas_locale.cpp index 851b687c4b..bb4fe93ba3 100644 --- a/xfa/fgas/localization/fgas_locale.cpp +++ b/xfa/fgas/localization/fgas_locale.cpp @@ -2304,12 +2304,12 @@ static FX_BOOL FX_ParseLocaleDate(const CFX_WideString& wsDate, continue; } uint32_t dwSymbolNum = 1; - uint32_t dwSymbol = strf[ccf++]; - while (ccf < lenf && strf[ccf] == dwSymbol) { + FX_WCHAR dwCharSymbol = strf[ccf++]; + while (ccf < lenf && strf[ccf] == dwCharSymbol) { ccf++; dwSymbolNum++; } - dwSymbol = (dwSymbol << 8) | (dwSymbolNum + '0'); + uint32_t dwSymbol = (dwCharSymbol << 8) | (dwSymbolNum + '0'); if (dwSymbol == FXBSTR_ID(0, 0, 'D', '1')) { if (!FX_IsDigit(str[cc])) { return FALSE; @@ -2523,12 +2523,12 @@ static FX_BOOL FX_ParseLocaleTime(const CFX_WideString& wsTime, continue; } uint32_t dwSymbolNum = 1; - uint32_t dwSymbol = strf[ccf++]; - while (ccf < lenf && strf[ccf] == dwSymbol) { + FX_WCHAR dwCharSymbol = strf[ccf++]; + while (ccf < lenf && strf[ccf] == dwCharSymbol) { ccf++; dwSymbolNum++; } - dwSymbol = (dwSymbol << 8) | (dwSymbolNum + '0'); + uint32_t dwSymbol = (dwCharSymbol << 8) | (dwSymbolNum + '0'); if (dwSymbol == FXBSTR_ID(0, 0, 'k', '1') || dwSymbol == FXBSTR_ID(0, 0, 'H', '1') || dwSymbol == FXBSTR_ID(0, 0, 'h', '1') || @@ -3952,12 +3952,12 @@ static FX_BOOL FX_DateFormat(const CFX_WideString& wsDatePattern, continue; } uint32_t dwSymbolNum = 1; - uint32_t dwSymbol = strf[ccf++]; - while (ccf < lenf && strf[ccf] == dwSymbol) { + FX_WCHAR dwCharSymbol = strf[ccf++]; + while (ccf < lenf && strf[ccf] == dwCharSymbol) { ccf++; dwSymbolNum++; } - dwSymbol = (dwSymbol << 8) | (dwSymbolNum + '0'); + uint32_t dwSymbol = (dwCharSymbol << 8) | (dwSymbolNum + '0'); if (dwSymbol == FXBSTR_ID(0, 0, 'D', '1')) { CFX_WideString wsDay; wsDay.Format(L"%d", day); @@ -4079,12 +4079,12 @@ static FX_BOOL FX_TimeFormat(const CFX_WideString& wsTimePattern, continue; } uint32_t dwSymbolNum = 1; - uint32_t dwSymbol = strf[ccf++]; - while (ccf < lenf && strf[ccf] == dwSymbol) { + FX_WCHAR dwCharSymbol = strf[ccf++]; + while (ccf < lenf && strf[ccf] == dwCharSymbol) { ccf++; dwSymbolNum++; } - dwSymbol = (dwSymbol << 8) | (dwSymbolNum + '0'); + uint32_t dwSymbol = (dwCharSymbol << 8) | (dwSymbolNum + '0'); if (dwSymbol == FXBSTR_ID(0, 0, 'h', '1')) { if (wHour > 12) { wHour -= 12; |