diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-04-09 20:12:05 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-09 20:12:05 +0000 |
| commit | 124abf5d68b9fe96bf8a95f239b77fdd47fdc556 (patch) | |
| tree | be5f7496c81c12dc593798e9769550836ffc1324 /xfa/fxfa/parser/cxfa_simple_parser.cpp | |
| parent | 741d5e1e0a64b85717ad054d3fd8807c66845a50 (diff) | |
| download | pdfium-124abf5d68b9fe96bf8a95f239b77fdd47fdc556.tar.xz | |
Merge CXFA_DocumentParser::{Start|Do}Parse methods
This CL combines the StartParse and DoParse methods of
CXFA_DocumentParser into a single Parse method since they are always
called together.
Change-Id: I45f38b2730f47c4719ca4f54c32ee8db6f11470d
Reviewed-on: https://pdfium-review.googlesource.com/29991
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_simple_parser.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_simple_parser.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/cxfa_simple_parser.cpp b/xfa/fxfa/parser/cxfa_simple_parser.cpp index de9f74f291..a59ea9d951 100644 --- a/xfa/fxfa/parser/cxfa_simple_parser.cpp +++ b/xfa/fxfa/parser/cxfa_simple_parser.cpp @@ -339,9 +339,8 @@ void CXFA_SimpleParser::SetFactory(CXFA_Document* pFactory) { m_pFactory = pFactory; } -int32_t CXFA_SimpleParser::StartParse( - const RetainPtr<IFX_SeekableStream>& pStream, - XFA_PacketType ePacketID) { +void CXFA_SimpleParser::StartParse(const RetainPtr<IFX_SeekableStream>& pStream, + XFA_PacketType ePacketID) { CloseParser(); m_pFileRead = pStream; m_pStream = pdfium::MakeRetain<CFX_SeekableStreamProxy>(pStream, false); @@ -355,7 +354,6 @@ int32_t CXFA_SimpleParser::StartParse( m_bParseStarted = true; m_ePacketID = ePacketID; - return XFA_PARSESTATUS_Ready; } int32_t CXFA_SimpleParser::DoParse() { |