diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-06-12 19:26:15 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-12 19:26:15 +0000 |
| commit | 8128485d0169a8dcbe14289a868ff3c27440bfcf (patch) | |
| tree | fd80bed666b8861efe40396bb599c948eb8c3c91 /fpdfsdk/fpdf_dataavail.cpp | |
| parent | a3843c99e7a631b2fbf87bbc12834e61bfb7b6f2 (diff) | |
| download | pdfium-8128485d0169a8dcbe14289a868ff3c27440bfcf.tar.xz | |
CheckUnSupportError() always called with FPDF_ERR_SUCCESS.
So remove the argument, and the logic dependent upon it being otherwise.
Rename function to more accurately reflect its purpose. Remove needless
null check while we're at it.
Change-Id: I7e119429fd159334eabcc250b5dac215580882ea
Reviewed-on: https://pdfium-review.googlesource.com/34950
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/fpdf_dataavail.cpp')
| -rw-r--r-- | fpdfsdk/fpdf_dataavail.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fpdfsdk/fpdf_dataavail.cpp b/fpdfsdk/fpdf_dataavail.cpp index d7bcf8f5c7..1b5fbeb35d 100644 --- a/fpdfsdk/fpdf_dataavail.cpp +++ b/fpdfsdk/fpdf_dataavail.cpp @@ -162,7 +162,7 @@ FPDFAvail_GetDocument(FPDF_AVAIL avail, FPDF_BYTESTRING password) { document->SetExtension(pdfium::MakeUnique<CPDFXFA_Context>(document.get())); #endif // PDF_ENABLE_XFA - CheckUnSupportError(document.get(), FPDF_ERR_SUCCESS); + ReportUnsupportedFeatures(document.get()); return FPDFDocumentFromCPDFDocument(document.release()); } |