From 54bc568bb41c5c258e95c24ef9d7355dcdc20f9e Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 20 Nov 2015 14:57:23 -0800 Subject: Replace |CPWL_ComboBox::operator CPWL_Edit()| with explicit method. More casts go away. R=thestig@chromium.org Review URL: https://codereview.chromium.org/1467653002 . --- fpdfsdk/src/formfiller/FFL_ComboBox.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'fpdfsdk/src') diff --git a/fpdfsdk/src/formfiller/FFL_ComboBox.cpp b/fpdfsdk/src/formfiller/FFL_ComboBox.cpp index ddaa519214..858cc6285a 100644 --- a/fpdfsdk/src/formfiller/FFL_ComboBox.cpp +++ b/fpdfsdk/src/formfiller/FFL_ComboBox.cpp @@ -137,8 +137,8 @@ void CFFL_ComboBox::GetActionData(CPDFSDK_PageView* pPageView, switch (type) { case CPDF_AAction::KeyStroke: if (CPWL_ComboBox* pComboBox = - (CPWL_ComboBox*)GetPDFWindow(pPageView, FALSE)) { - if (CPWL_Edit* pEdit = (CPWL_Edit*)*pComboBox) { + static_cast(GetPDFWindow(pPageView, FALSE))) { + if (CPWL_Edit* pEdit = pComboBox->GetEdit()) { fa.bFieldFull = pEdit->IsTextFull(); int nSelStart = 0; int nSelEnd = 0; @@ -157,8 +157,8 @@ void CFFL_ComboBox::GetActionData(CPDFSDK_PageView* pPageView, break; case CPDF_AAction::Validate: if (CPWL_ComboBox* pComboBox = - (CPWL_ComboBox*)GetPDFWindow(pPageView, FALSE)) { - if (CPWL_Edit* pEdit = (CPWL_Edit*)*pComboBox) { + static_cast(GetPDFWindow(pPageView, FALSE))) { + if (CPWL_Edit* pEdit = pComboBox->GetEdit()) { fa.sValue = pEdit->GetText(); } } @@ -179,8 +179,8 @@ void CFFL_ComboBox::SetActionData(CPDFSDK_PageView* pPageView, switch (type) { case CPDF_AAction::KeyStroke: if (CPWL_ComboBox* pComboBox = - (CPWL_ComboBox*)GetPDFWindow(pPageView, FALSE)) { - if (CPWL_Edit* pEdit = (CPWL_Edit*)*pComboBox) { + static_cast(GetPDFWindow(pPageView, FALSE))) { + if (CPWL_Edit* pEdit = pComboBox->GetEdit()) { pEdit->SetSel(fa.nSelStart, fa.nSelEnd); pEdit->ReplaceSel(fa.sChange.c_str()); } @@ -210,10 +210,10 @@ void CFFL_ComboBox::SaveState(CPDFSDK_PageView* pPageView) { ASSERT(pPageView != NULL); if (CPWL_ComboBox* pComboBox = - (CPWL_ComboBox*)GetPDFWindow(pPageView, FALSE)) { + static_cast(GetPDFWindow(pPageView, FALSE))) { m_State.nIndex = pComboBox->GetSelect(); - if (CPWL_Edit* pEdit = (CPWL_Edit*)*pComboBox) { + if (CPWL_Edit* pEdit = pComboBox->GetEdit()) { pEdit->GetSel(m_State.nStart, m_State.nEnd); m_State.sValue = pEdit->GetText(); } @@ -224,11 +224,11 @@ void CFFL_ComboBox::RestoreState(CPDFSDK_PageView* pPageView) { ASSERT(pPageView != NULL); if (CPWL_ComboBox* pComboBox = - (CPWL_ComboBox*)GetPDFWindow(pPageView, TRUE)) { + static_cast(GetPDFWindow(pPageView, TRUE))) { if (m_State.nIndex >= 0) pComboBox->SetSelect(m_State.nIndex); else { - if (CPWL_Edit* pEdit = (CPWL_Edit*)*pComboBox) { + if (CPWL_Edit* pEdit = pComboBox->GetEdit()) { pEdit->SetText(m_State.sValue.c_str()); pEdit->SetSel(m_State.nStart, m_State.nEnd); } -- cgit v1.2.3