From d1a8458e6390103e123e9d265040b3d02c16955b Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Wed, 2 Aug 2017 17:41:22 -0700 Subject: Avoid a redundant header offset check in CPDF_Parser. CPDF_Parser::StartLinearizedParse() calls StartParse(), but already knows the PDF header offset. Refactor StartParse() so it does not have to look for the header again. Change-Id: Id8cc39301ae72da868dafc53921622d5b28ce26e Reviewed-on: https://pdfium-review.googlesource.com/9830 Commit-Queue: Lei Zhang Reviewed-by: Art Snake --- core/fpdfapi/parser/cpdf_parser.h | 3 +++ 1 file changed, 3 insertions(+) (limited to 'core/fpdfapi/parser/cpdf_parser.h') diff --git a/core/fpdfapi/parser/cpdf_parser.h b/core/fpdfapi/parser/cpdf_parser.h index 759d042360..189de168f4 100644 --- a/core/fpdfapi/parser/cpdf_parser.h +++ b/core/fpdfapi/parser/cpdf_parser.h @@ -153,6 +153,9 @@ class CPDF_Parser { ObjectInfo info; }; + Error StartParseInternal(const CFX_RetainPtr& pFile, + CPDF_Document* pDocument, + int32_t iHeaderOffset); CPDF_Object* ParseDirect(CPDF_Object* pObj); bool LoadAllCrossRefV4(FX_FILESIZE pos); bool LoadAllCrossRefV5(FX_FILESIZE pos); -- cgit v1.2.3