diff options
| author | Tom Sepez <tsepez@chromium.org> | 2016-01-26 13:28:23 -0800 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2016-01-26 13:28:23 -0800 |
| commit | 0ec8bb4486ab315283f36d071be2ddc1b80998c3 (patch) | |
| tree | ece6793f078348acb3ce9c6dd55ab0b109ed130c /core/src/fpdfapi/fpdf_parser | |
| parent | 66a9dc9dd65ed2b9c7ede448d046b770140f16d5 (diff) | |
| download | pdfium-0ec8bb4486ab315283f36d071be2ddc1b80998c3.tar.xz | |
Move struct ContentParam off of CPDF_Object's enums
They mean different things. The goal is that only usage of
the CPDF_Object type enums outside of the class should be for
switch() statements, so remove one other comparison.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1635173003 .
Diffstat (limited to 'core/src/fpdfapi/fpdf_parser')
| -rw-r--r-- | core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp index c0150b5435..8c2bad4acd 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp @@ -4851,7 +4851,7 @@ FX_BOOL CPDF_HintTables::LoadHintStream(CPDF_Stream* pHintStream) { return FALSE; CPDF_Dictionary* pDict = pHintStream->GetDict(); CPDF_Object* pOffset = pDict ? pDict->GetElement("S") : nullptr; - if (!pOffset || pOffset->GetType() != CPDF_Object::NUMBER) + if (!pOffset || !pOffset->IsNumber()) return FALSE; int shared_hint_table_offset = pOffset->GetInteger(); CPDF_StreamAcc acc; |