diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-02-24 14:07:17 -0800 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-02-27 15:58:45 +0000 |
| commit | 717a4fc857d66017cecc4c8f8285713135b9dc68 (patch) | |
| tree | 55816996a4907d87cacc02b4966d977d7629139d /core | |
| parent | 1d7018e92d315c883dca38dd083db44791761827 (diff) | |
| download | pdfium-717a4fc857d66017cecc4c8f8285713135b9dc68.tar.xz | |
Remove some pointless checks and assignments.
Change-Id: I6f9fd67cdac581e8f91f829a5934b763bb229d21
Reviewed-on: https://pdfium-review.googlesource.com/2838
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_data_avail.cpp | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/core/fpdfapi/parser/cpdf_data_avail.cpp b/core/fpdfapi/parser/cpdf_data_avail.cpp index c6ecae9b1d..64eaf2a43b 100644 --- a/core/fpdfapi/parser/cpdf_data_avail.cpp +++ b/core/fpdfapi/parser/cpdf_data_avail.cpp @@ -1124,13 +1124,8 @@ bool CPDF_DataAvail::CheckArrayPageNode(uint32_t dwPageNo, return false; } - if (!pPages) { - if (m_docStatus == PDF_DATAAVAIL_ERROR) { - m_docStatus = PDF_DATAAVAIL_ERROR; - return false; - } + if (!pPages) return false; - } CPDF_Array* pArray = pPages->AsArray(); if (!pArray) { @@ -1161,11 +1156,8 @@ bool CPDF_DataAvail::CheckUnknownPageNode(uint32_t dwPageNo, return false; } - if (!pPage) { - if (m_docStatus == PDF_DATAAVAIL_ERROR) - m_docStatus = PDF_DATAAVAIL_ERROR; + if (!pPage) return false; - } if (pPage->IsArray()) { pPageNode->m_dwPageNo = dwPageNo; |