From 9c98adb0404c4b161ceb71953772b79983afa55d Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 5 Feb 2016 12:45:58 -0800 Subject: Fix build broken at e059b5ba1260 TBR=thestig@chromium.org Review URL: https://codereview.chromium.org/1671213002 . --- xfa/src/fwl/src/lightwidget/combobox.cpp | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/xfa/src/fwl/src/lightwidget/combobox.cpp b/xfa/src/fwl/src/lightwidget/combobox.cpp index 07556b7797..e6c0160b34 100644 --- a/xfa/src/fwl/src/lightwidget/combobox.cpp +++ b/xfa/src/fwl/src/lightwidget/combobox.cpp @@ -44,9 +44,10 @@ int32_t CFWL_ComboBox::AddString(const CFX_WideStringC& wsText, return m_comboBoxData.m_ItemArray.size() - 1; } bool CFWL_ComboBox::RemoveAt(int32_t iIndex) { - if (iIndex < 0 || iIndex >= m_comboBoxData.m_ItemArray.size()) + if (iIndex < 0 || + static_cast(iIndex) >= m_comboBoxData.m_ItemArray.size()) { return false; - + } m_comboBoxData.m_ItemArray.erase(m_comboBoxData.m_ItemArray.begin() + iIndex); return true; } @@ -245,9 +246,10 @@ int32_t CFWL_ComboBox::CFWL_ComboBoxDP::CountItems(IFWL_Widget* pWidget) { } FWL_HLISTITEM CFWL_ComboBox::CFWL_ComboBoxDP::GetItem(IFWL_Widget* pWidget, int32_t nIndex) { - return nIndex >= 0 && nIndex < m_ItemArray.size() - ? reinterpret_cast(m_ItemArray[nIndex].get()) - : nullptr; + if (nIndex < 0 || static_cast(nIndex) >= m_ItemArray.size()) + return nullptr; + + return reinterpret_cast(m_ItemArray[nIndex].get()); } int32_t CFWL_ComboBox::CFWL_ComboBoxDP::GetItemIndex(IFWL_Widget* pWidget, FWL_HLISTITEM hItem) { @@ -261,7 +263,7 @@ int32_t CFWL_ComboBox::CFWL_ComboBoxDP::GetItemIndex(IFWL_Widget* pWidget, FX_BOOL CFWL_ComboBox::CFWL_ComboBoxDP::SetItemIndex(IFWL_Widget* pWidget, FWL_HLISTITEM hItem, int32_t nIndex) { - if (nIndex < 0 || nIndex >= m_ItemArray.size()) + if (nIndex < 0 || static_cast(nIndex) >= m_ItemArray.size()) return FALSE; m_ItemArray[nIndex].reset(reinterpret_cast(hItem)); -- cgit v1.2.3