summaryrefslogtreecommitdiff
path: root/fpdfsdk/fpdfview.cpp
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2018-01-29 19:01:26 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-29 19:01:26 +0000
commitd6e92bd6e874a4709144effc79163ef3c13d4b2e (patch)
treec0d08778d3427dd1e0283166d2d87107fe8f8da4 /fpdfsdk/fpdfview.cpp
parent2f7a01849e698b3d63e79d7e8bf04a6346fa248e (diff)
downloadpdfium-d6e92bd6e874a4709144effc79163ef3c13d4b2e.tar.xz
Clean up FPDF_GetFormType().
- Update public API documentation. - Fix nits in the implementation. Change-Id: I4c7af07b0ec178a5986f45652663f48491b7d078 Reviewed-on: https://pdfium-review.googlesource.com/24330 Reviewed-by: Ryan Harrison <rharrison@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fpdfsdk/fpdfview.cpp')
-rw-r--r--fpdfsdk/fpdfview.cpp7
1 files changed, 2 insertions, 5 deletions
diff --git a/fpdfsdk/fpdfview.cpp b/fpdfsdk/fpdfview.cpp
index 97fc02a234..e890aa0090 100644
--- a/fpdfsdk/fpdfview.cpp
+++ b/fpdfsdk/fpdfview.cpp
@@ -574,9 +574,6 @@ FPDF_LoadDocument(FPDF_STRING file_path, FPDF_BYTESTRING password) {
}
FPDF_EXPORT int FPDF_CALLCONV FPDF_GetFormType(FPDF_DOCUMENT document) {
- if (!document)
- return false;
-
const CPDF_Document* pDoc = CPDFDocumentFromFPDFDocument(document);
if (!pDoc)
return FORMTYPE_NONE;
@@ -593,8 +590,8 @@ FPDF_EXPORT int FPDF_CALLCONV FPDF_GetFormType(FPDF_DOCUMENT document) {
if (!pXFA)
return FORMTYPE_ACRO_FORM;
- bool needsRendering = pRoot->GetBooleanFor("NeedsRendering", false);
- return needsRendering ? FORMTYPE_XFA_FULL : FORMTYPE_XFA_FOREGROUND;
+ bool bNeedsRendering = pRoot->GetBooleanFor("NeedsRendering", false);
+ return bNeedsRendering ? FORMTYPE_XFA_FULL : FORMTYPE_XFA_FOREGROUND;
}
#ifdef PDF_ENABLE_XFA