diff options
author | thestig <thestig@chromium.org> | 2016-06-23 19:57:45 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-06-23 19:57:45 -0700 |
commit | 29d447ae35101675a2a2d8bc1dcfca65de7f3929 (patch) | |
tree | 6a84b11413a53fb547e242bea6a5ddf55df7293a /core/fpdfapi/fpdf_parser/include | |
parent | ce56557ef58facf2519f541d5cad33ea121b4c21 (diff) | |
download | pdfium-29d447ae35101675a2a2d8bc1dcfca65de7f3929.tar.xz |
Improve hint table validation checks.
Check required hint table dictionary entries and make sure they:
- Exist.
- Are of the right type.
Along the way:
- Fix FX_atonum() to not have a non-const pass-by-ref param.
- Simplify code in CPDF_StreamContentParser.
- Make CPDF_Number::IsInteger() a const method.
BUG=610555
Review-Url: https://codereview.chromium.org/2095763003
Diffstat (limited to 'core/fpdfapi/fpdf_parser/include')
-rw-r--r-- | core/fpdfapi/fpdf_parser/include/cpdf_number.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fpdfapi/fpdf_parser/include/cpdf_number.h b/core/fpdfapi/fpdf_parser/include/cpdf_number.h index 7b93a992b1..eaa4009543 100644 --- a/core/fpdfapi/fpdf_parser/include/cpdf_number.h +++ b/core/fpdfapi/fpdf_parser/include/cpdf_number.h @@ -29,12 +29,12 @@ class CPDF_Number : public CPDF_Object { CPDF_Number* AsNumber() override; const CPDF_Number* AsNumber() const override; - FX_BOOL IsInteger() { return m_bInteger; } + bool IsInteger() const { return m_bInteger; } protected: ~CPDF_Number() override; - FX_BOOL m_bInteger; + bool m_bInteger; union { int m_Integer; FX_FLOAT m_Float; |