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_dataimporter.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_dataimporter.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_dataimporter.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/cxfa_dataimporter.cpp b/xfa/fxfa/parser/cxfa_dataimporter.cpp index 7c6e0ec12d..c5c23fcfb9 100644 --- a/xfa/fxfa/parser/cxfa_dataimporter.cpp +++ b/xfa/fxfa/parser/cxfa_dataimporter.cpp @@ -28,10 +28,7 @@ bool CXFA_DataImporter::ImportData( const RetainPtr<IFX_SeekableStream>& pDataDocument) { auto pDataDocumentParser = pdfium::MakeUnique<CXFA_SimpleParser>(m_pDocument.Get()); - if (pDataDocumentParser->StartParse( - pDataDocument, XFA_PacketType::Datasets) != XFA_PARSESTATUS_Ready) { - return false; - } + pDataDocumentParser->StartParse(pDataDocument, XFA_PacketType::Datasets); if (pDataDocumentParser->DoParse() < XFA_PARSESTATUS_Done) return false; |