diff options
| author | dan sinclair <dsinclair@chromium.org> | 2018-03-08 15:19:50 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-03-08 15:19:50 +0000 |
| commit | 7544a4b4fd98b4cf6e06e717758e237ffd822671 (patch) | |
| tree | f2f7ee9cb7dc25eeda4e5c0d87ccf713111327a1 /fpdfsdk/cpdfsdk_interform.cpp | |
| parent | 507fb4ed09d5fbc92fafdfe405d79d44d11a4664 (diff) | |
| download | pdfium-7544a4b4fd98b4cf6e06e717758e237ffd822671.tar.xz | |
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 <rharrison@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_interform.cpp')
| -rw-r--r-- | fpdfsdk/cpdfsdk_interform.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
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<CPDF_Object*> fieldObjects = af.GetAllFields(); std::vector<CPDF_FormField*> fields = GetFieldFromObjects(fieldObjects); - return m_pInterForm->ResetForm(fields, !(dwFlags & 0x01), true); + m_pInterForm->ResetForm(fields, !(dwFlags & 0x01), true); } std::vector<CPDF_FormField*> CPDFSDK_InterForm::GetFieldFromObjects( |