diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-10-24 17:26:50 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-24 17:26:50 +0000 |
| commit | 2237541071da7f309c16b3033daa4c6be170dca4 (patch) | |
| tree | a468618a44666558fae3432ece2d3439a0ac2537 /fpdfsdk/fpdf_edittext.cpp | |
| parent | d86805176f390e0fec1802aae7dbbf1d1d9f53b0 (diff) | |
| download | pdfium-2237541071da7f309c16b3033daa4c6be170dca4.tar.xz | |
Check CPDFFooObjectFromFPDFPageObject() result.
Instead of checking the FPDF_OBJECT itself. Remove redundant checks and
fix nits.
Change-Id: Id566847d39d16331c08f0b5f3b46431716717b5c
Reviewed-on: https://pdfium-review.googlesource.com/c/44535
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fpdfsdk/fpdf_edittext.cpp')
| -rw-r--r-- | fpdfsdk/fpdf_edittext.cpp | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/fpdfsdk/fpdf_edittext.cpp b/fpdfsdk/fpdf_edittext.cpp index 32ea6db607..1c6709f915 100644 --- a/fpdfsdk/fpdf_edittext.cpp +++ b/fpdfsdk/fpdf_edittext.cpp @@ -452,11 +452,7 @@ FPDFPageObj_NewTextObj(FPDF_DOCUMENT document, FPDF_EXPORT FPDF_BOOL FPDF_CALLCONV FPDFText_SetText(FPDF_PAGEOBJECT text_object, FPDF_WIDESTRING text) { - if (!text_object) - return false; - - CPDF_TextObject* pTextObj = - CPDFPageObjectFromFPDFPageObject(text_object)->AsText(); + CPDF_TextObject* pTextObj = CPDFTextObjectFromFPDFPageObject(text_object); if (!pTextObj) return false; @@ -614,9 +610,6 @@ FPDFPageObj_CreateTextObj(FPDF_DOCUMENT document, } FPDF_EXPORT int FPDF_CALLCONV FPDFText_GetTextRenderMode(FPDF_PAGEOBJECT text) { - if (!text) - return -1; - CPDF_TextObject* pTextObj = CPDFTextObjectFromFPDFPageObject(text); if (!pTextObj) return -1; |