From 7544a4b4fd98b4cf6e06e717758e237ffd822671 Mon Sep 17 00:00:00 2001 From: dan sinclair Date: Thu, 8 Mar 2018 15:19:50 +0000 Subject: CPDF_InterForm::ResetForm always returns true This method always returns true, so remove the return value. Change-Id: I3da93fc5face39a53b589787873839c06c9fcfab Reviewed-on: https://pdfium-review.googlesource.com/28210 Commit-Queue: Ryan Harrison Reviewed-by: Ryan Harrison --- fpdfsdk/cpdfsdk_interform.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'fpdfsdk/cpdfsdk_interform.cpp') diff --git a/fpdfsdk/cpdfsdk_interform.cpp b/fpdfsdk/cpdfsdk_interform.cpp index 57f3a6e90e..f4e22d7744 100644 --- a/fpdfsdk/cpdfsdk_interform.cpp +++ b/fpdfsdk/cpdfsdk_interform.cpp @@ -592,19 +592,21 @@ ByteString CPDFSDK_InterForm::ExportFormToFDFTextBuf() { return pFDF ? pFDF->WriteToString() : ByteString(); } -bool CPDFSDK_InterForm::DoAction_ResetForm(const CPDF_Action& action) { +void CPDFSDK_InterForm::DoAction_ResetForm(const CPDF_Action& action) { ASSERT(action.GetDict()); CPDF_Dictionary* pActionDict = action.GetDict(); - if (!pActionDict->KeyExist("Fields")) - return m_pInterForm->ResetForm(true); + if (!pActionDict->KeyExist("Fields")) { + m_pInterForm->ResetForm(true); + return; + } CPDF_ActionFields af(&action); uint32_t dwFlags = action.GetFlags(); std::vector fieldObjects = af.GetAllFields(); std::vector fields = GetFieldFromObjects(fieldObjects); - return m_pInterForm->ResetForm(fields, !(dwFlags & 0x01), true); + m_pInterForm->ResetForm(fields, !(dwFlags & 0x01), true); } std::vector CPDFSDK_InterForm::GetFieldFromObjects( -- cgit v1.2.3