diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-01-27 09:47:45 -0800 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-01-27 09:47:45 -0800 |
| commit | eef005055d6aafc2cc6066af37ce47d8b15ec7cd (patch) | |
| tree | ab47aecac44e920d06b0d5e5aac6e15ad316a827 /core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp | |
| parent | a03932372b0906a340a6e3860c87e45f9ec79042 (diff) | |
| download | pdfium-chromium/2292.tar.xz | |
Kill scattered extern _PDF_CharType declarations.chromium/2292chromium/2291chromium/2290
While we're at it, avoid an indirection through a pointer, and
use a name that isn't reserved for the compiler (leading _ CAP).
This is a small portion of the associated bug:
BUG=https://code.google.com/p/pdfium/issues/detail?id=112
R=brucedawson@chromium.org
Review URL: https://codereview.chromium.org/880663003
Diffstat (limited to 'core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp')
| -rw-r--r-- | core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp index 610fb39781..d17b123765 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_filters.cpp @@ -576,12 +576,11 @@ CPDF_Ascii85Filter::CPDF_Ascii85Filter() m_State = 0; m_CharCount = 0; } -extern const FX_LPCSTR _PDF_CharType; void CPDF_Ascii85Filter::v_FilterIn(FX_LPCBYTE src_buf, FX_DWORD src_size, CFX_BinaryBuf& dest_buf) { for (FX_DWORD i = 0; i < src_size; i ++) { FX_BYTE byte = src_buf[i]; - if (_PDF_CharType[byte] == 'W') { + if (PDF_CharType[byte] == 'W') { continue; } switch (m_State) { @@ -640,7 +639,7 @@ void CPDF_AsciiHexFilter::v_FilterIn(FX_LPCBYTE src_buf, FX_DWORD src_size, CFX_ { for (FX_DWORD i = 0; i < src_size; i ++) { FX_BYTE byte = src_buf[i]; - if (_PDF_CharType[byte] == 'W') { + if (PDF_CharType[byte] == 'W') { continue; } int digit; |