accessibility/inc/standard/vclxaccessibletoolbox.hxx | 15 - accessibility/source/standard/vclxaccessibletoolbox.cxx | 84 +++--- accessibility/source/standard/vclxaccessibletoolboxitem.cxx | 6 cui/source/customize/cfg.cxx | 19 - extensions/source/bibliography/toolbar.cxx | 4 forms/source/solar/control/navtoolbar.cxx | 16 - framework/inc/uielement/toolbarmerger.hxx | 14 - framework/source/uielement/addonstoolbarmanager.cxx | 10 framework/source/uielement/toolbarmanager.cxx | 19 - framework/source/uielement/toolbarmerger.cxx | 34 +- include/sfx2/sidebar/SidebarToolBox.hxx | 2 include/vcl/toolbox.hxx | 56 ++-- reportdesign/source/ui/dlg/AddField.cxx | 10 reportdesign/source/ui/dlg/Condition.cxx | 4 reportdesign/source/ui/misc/toolboxcontroller.cxx | 4 sfx2/source/sidebar/SidebarToolBox.cxx | 2 sfx2/source/view/viewsh.cxx | 4 svtools/source/uno/toolboxcontroller.cxx | 4 svx/source/dialog/srchdlg.cxx | 2 svx/source/tbxctrls/tbunosearchcontrollers.cxx | 12 sw/source/uibase/ribbar/workctrl.cxx | 2 vcl/inc/toolbox.h | 2 vcl/osx/a11yfocustracker.cxx | 5 vcl/source/window/builder.cxx | 1 vcl/source/window/menubarwindow.cxx | 8 vcl/source/window/toolbox.cxx | 144 +++++------ vcl/source/window/toolbox2.cxx | 151 ++++++------ vcl/unx/gtk/a11y/atkutil.cxx | 5 28 files changed, 335 insertions(+), 304 deletions(-)
New commits: commit 6e4d3bad807a32371fe49f0a4a28ca0e5b0f6e70 Author: Stephan Bergmann <sberg...@redhat.com> Date: Mon Mar 13 13:27:46 2017 +0100 Clean up integer types representing positions in ToolBox's item vector (vs. uses of sal_uInt16 representing such items' IDs). APPEND and ITEM_NOTFOUND are members of ToolBox now (and have been bumped to the corresponding type's max). The payload of VclEventIds ToolboxButtonStateChanged ToolboxClick ToolboxHighlightOff ToolboxItemAdded ToolboxItemDisabled ToolboxItemEnabled ToolboxItemRemoved ToolboxItemTextChanged ToolboxItemUpdated ToolboxItemWindowChanged ToolboxSelect has been changed too (note that reading/writing of those payloads isn't very consistent, though: for some of these, values are written that are never read, while for some others no values are ever written but the reading code would like to receive some value if the payload wasn't a nullptr). Change-Id: I4856ffb3637404b8d866621a3503d7922b1b8141 diff --git a/accessibility/inc/standard/vclxaccessibletoolbox.hxx b/accessibility/inc/standard/vclxaccessibletoolbox.hxx index f914075..b84add9 100644 --- a/accessibility/inc/standard/vclxaccessibletoolbox.hxx +++ b/accessibility/inc/standard/vclxaccessibletoolbox.hxx @@ -21,6 +21,7 @@ #include <map> #include <toolkit/awt/vclxaccessiblecomponent.hxx> +#include <vcl/toolbox.hxx> #include <com/sun/star/accessibility/XAccessibleSelection.hpp> @@ -39,16 +40,16 @@ class VCLXAccessibleToolBox : public VCLXAccessibleComponent, public VCLXAccessi private: ToolBoxItemsMap m_aAccessibleChildren; - VCLXAccessibleToolBoxItem* GetItem_Impl( sal_Int32 _nPos ); + VCLXAccessibleToolBoxItem* GetItem_Impl( ToolBox::ImplToolItems::size_type _nPos ); void UpdateFocus_Impl(); - void ReleaseFocus_Impl( sal_Int32 _nPos ); - void UpdateChecked_Impl( sal_Int32 _nPos ); - void UpdateIndeterminate_Impl( sal_Int32 _nPos ); - void UpdateItem_Impl( sal_Int32 _nPos ); + void ReleaseFocus_Impl( ToolBox::ImplToolItems::size_type _nPos ); + void UpdateChecked_Impl( ToolBox::ImplToolItems::size_type _nPos ); + void UpdateIndeterminate_Impl( ToolBox::ImplToolItems::size_type _nPos ); + void UpdateItem_Impl( ToolBox::ImplToolItems::size_type _nPos ); void UpdateAllItems_Impl(); - void UpdateItemName_Impl( sal_Int32 _nPos ); - void UpdateItemEnabled_Impl( sal_Int32 _nPos ); + void UpdateItemName_Impl( ToolBox::ImplToolItems::size_type _nPos ); + void UpdateItemEnabled_Impl( ToolBox::ImplToolItems::size_type _nPos ); void UpdateCustomPopupItemp_Impl( vcl::Window* pWindow, bool bOpen ); void HandleSubToolBarEvent( const VclWindowEvent& rVclWindowEvent ); void ReleaseSubToolBox( ToolBox* _pSubToolBox ); diff --git a/accessibility/source/standard/vclxaccessibletoolbox.cxx b/accessibility/source/standard/vclxaccessibletoolbox.cxx index c4cd2cb..aa4cad6 100644 --- a/accessibility/source/standard/vclxaccessibletoolbox.cxx +++ b/accessibility/source/standard/vclxaccessibletoolbox.cxx @@ -182,13 +182,14 @@ VCLXAccessibleToolBox::~VCLXAccessibleToolBox() { } -VCLXAccessibleToolBoxItem* VCLXAccessibleToolBox::GetItem_Impl( sal_Int32 _nPos ) +VCLXAccessibleToolBoxItem* VCLXAccessibleToolBox::GetItem_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VCLXAccessibleToolBoxItem* pItem = nullptr; VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) { ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.find( _nPos ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! // returns only toolbox buttons, not windows if ( aIter != m_aAccessibleChildren.end() && aIter->second.is()) pItem = static_cast< VCLXAccessibleToolBoxItem* >( aIter->second.get() ); @@ -223,7 +224,7 @@ void VCLXAccessibleToolBox::UpdateFocus_Impl() for ( ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.begin(); aIter != m_aAccessibleChildren.end(); ++aIter ) { - sal_uInt16 nItemId = pToolBox->GetItemId( (sal_uInt16)aIter->first ); + sal_uInt16 nItemId = pToolBox->GetItemId( aIter->first ); if ( aIter->second.is() ) { @@ -249,12 +250,13 @@ void VCLXAccessibleToolBox::UpdateFocus_Impl() } } -void VCLXAccessibleToolBox::ReleaseFocus_Impl( sal_Int32 _nPos ) +void VCLXAccessibleToolBox::ReleaseFocus_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) // #107124#, do not check for focus because this message is also handled in losefocus { ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.find( _nPos ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! if ( aIter != m_aAccessibleChildren.end() && aIter->second.is() ) { VCLXAccessibleToolBoxItem* pItem = @@ -265,18 +267,18 @@ void VCLXAccessibleToolBox::ReleaseFocus_Impl( sal_Int32 _nPos ) } } -void VCLXAccessibleToolBox::UpdateChecked_Impl( sal_Int32 _nPos ) +void VCLXAccessibleToolBox::UpdateChecked_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) { - sal_uInt16 nFocusId = pToolBox->GetItemId( (sal_uInt16)_nPos ); + sal_uInt16 nFocusId = pToolBox->GetItemId( _nPos ); VCLXAccessibleToolBoxItem* pFocusItem = nullptr; for ( ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.begin(); aIter != m_aAccessibleChildren.end(); ++aIter ) { - sal_uInt16 nItemId = pToolBox->GetItemId( (sal_uInt16)aIter->first ); + sal_uInt16 nItemId = pToolBox->GetItemId( aIter->first ); VCLXAccessibleToolBoxItem* pItem = static_cast< VCLXAccessibleToolBoxItem* >( aIter->second.get() ); @@ -285,19 +287,20 @@ void VCLXAccessibleToolBox::UpdateChecked_Impl( sal_Int32 _nPos ) pFocusItem = pItem; } //Solution:If the position is not a child item,the focus should not be called - if ( pFocusItem && (sal_uInt16)_nPos != TOOLBOX_ITEM_NOTFOUND ) + if ( pFocusItem && _nPos != ToolBox::ITEM_NOTFOUND ) pFocusItem->SetFocus( true ); } } -void VCLXAccessibleToolBox::UpdateIndeterminate_Impl( sal_Int32 _nPos ) +void VCLXAccessibleToolBox::UpdateIndeterminate_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) { - sal_uInt16 nItemId = pToolBox->GetItemId( (sal_uInt16)_nPos ); + sal_uInt16 nItemId = pToolBox->GetItemId( _nPos ); ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.find( _nPos ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! if ( aIter != m_aAccessibleChildren.end() && aIter->second.is() ) { VCLXAccessibleToolBoxItem* pItem = @@ -336,9 +339,9 @@ void VCLXAccessibleToolBox::implReleaseToolboxItem( ToolBoxItemsMap::iterator& _ } } -void VCLXAccessibleToolBox::UpdateItem_Impl( sal_Int32 _nPos) +void VCLXAccessibleToolBox::UpdateItem_Impl( ToolBox::ImplToolItems::size_type _nPos) { - if ( _nPos < sal_Int32( m_aAccessibleChildren.size() ) ) + if ( _nPos < m_aAccessibleChildren.size() ) { UpdateAllItems_Impl(); return; @@ -349,6 +352,7 @@ void VCLXAccessibleToolBox::UpdateItem_Impl( sal_Int32 _nPos) { // adjust the "index-in-parent"s ToolBoxItemsMap::iterator aIndexAdjust = m_aAccessibleChildren.upper_bound( _nPos ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! while ( m_aAccessibleChildren.end() != aIndexAdjust ) { Reference< XAccessible > xItemAcc( aIndexAdjust->second ); @@ -380,6 +384,7 @@ void VCLXAccessibleToolBox::UpdateItem_Impl( sal_Int32 _nPos) // TODO: we should make this dependent on the existence of event listeners // with the current implementation, we always create accessible object Any aNewChild = Any( getAccessibleChild( (sal_Int32)_nPos ) ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! NotifyAccessibleEvent( AccessibleEventId::CHILD, Any(), aNewChild ); } } @@ -398,11 +403,12 @@ void VCLXAccessibleToolBox::UpdateAllItems_Impl() m_aAccessibleChildren.clear(); // register the new items - sal_uInt16 i, nCount = pToolBox->GetItemCount(); + ToolBox::ImplToolItems::size_type i, nCount = pToolBox->GetItemCount(); for ( i = 0; i < nCount; ++i ) { Any aNewValue; aNewValue <<= getAccessibleChild( (sal_Int32)i ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! NotifyAccessibleEvent( AccessibleEventId::CHILD, Any(), aNewValue ); } } @@ -423,6 +429,7 @@ void VCLXAccessibleToolBox::UpdateCustomPopupItemp_Impl( vcl::Window* pWindow, b if( xChild.is() ) { Reference< XAccessible > xChildItem( getAccessibleChild( static_cast< sal_Int32 >( pToolBox->GetItemPos( nDownItem ) ) ) ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! VCLXAccessibleToolBoxItem* pItem = static_cast< VCLXAccessibleToolBoxItem* >( xChildItem.get() ); pItem->SetChild( xChild ); @@ -431,14 +438,14 @@ void VCLXAccessibleToolBox::UpdateCustomPopupItemp_Impl( vcl::Window* pWindow, b } } -void VCLXAccessibleToolBox::UpdateItemName_Impl( sal_Int32 _nPos ) +void VCLXAccessibleToolBox::UpdateItemName_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VCLXAccessibleToolBoxItem* pItem = GetItem_Impl( _nPos ); if ( pItem ) pItem->NameChanged(); } -void VCLXAccessibleToolBox::UpdateItemEnabled_Impl( sal_Int32 _nPos ) +void VCLXAccessibleToolBox::UpdateItemEnabled_Impl( ToolBox::ImplToolItems::size_type _nPos ) { VCLXAccessibleToolBoxItem* pItem = GetItem_Impl( _nPos ); if ( pItem ) @@ -454,8 +461,9 @@ void VCLXAccessibleToolBox::HandleSubToolBarEvent( const VclWindowEvent& rVclWin && pToolBox == pChildWindow->GetParent() && pChildWindow->GetType() == WindowType::TOOLBOX ) { - sal_Int32 nIndex = pToolBox->GetItemPos( pToolBox->GetCurItemId() ); + ToolBox::ImplToolItems::size_type nIndex = pToolBox->GetItemPos( pToolBox->GetCurItemId() ); Reference< XAccessible > xItem = getAccessibleChild( nIndex ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! if ( xItem.is() ) { Reference< XAccessible > xChild = pChildWindow->GetAccessible(); @@ -473,11 +481,12 @@ void VCLXAccessibleToolBox::ReleaseSubToolBox( ToolBox* _pSubToolBox ) if ( !pToolBox ) return; - sal_Int32 nIndex = pToolBox->GetItemPos( pToolBox->GetCurItemId() ); - if ( nIndex == SAL_MAX_UINT16 ) + ToolBox::ImplToolItems::size_type nIndex = pToolBox->GetItemPos( pToolBox->GetCurItemId() ); + if ( nIndex == ToolBox::ITEM_NOTFOUND ) return; // not found Reference< XAccessible > xItem = getAccessibleChild( nIndex ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! if ( !xItem.is() ) return; @@ -519,10 +528,10 @@ void VCLXAccessibleToolBox::ProcessWindowEvent( const VclWindowEvent& rVclWindow VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( rVclWindowEvent.GetData() ) { - UpdateChecked_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); - UpdateIndeterminate_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + UpdateChecked_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); + UpdateIndeterminate_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); } - else if( pToolBox->GetItemPos(pToolBox->GetCurItemId()) != TOOLBOX_ITEM_NOTFOUND ) + else if( pToolBox->GetItemPos(pToolBox->GetCurItemId()) != ToolBox::ITEM_NOTFOUND ) { UpdateChecked_Impl( pToolBox->GetItemPos(pToolBox->GetCurItemId()) ); UpdateIndeterminate_Impl( pToolBox->GetItemPos(pToolBox->GetCurItemId()) ); @@ -539,8 +548,8 @@ void VCLXAccessibleToolBox::ProcessWindowEvent( const VclWindowEvent& rVclWindow { if ( rVclWindowEvent.GetData() ) { - UpdateChecked_Impl( TOOLBOX_ITEM_NOTFOUND ); - UpdateIndeterminate_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + UpdateChecked_Impl( ToolBox::ITEM_NOTFOUND ); + UpdateIndeterminate_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); } break; } @@ -550,11 +559,11 @@ void VCLXAccessibleToolBox::ProcessWindowEvent( const VclWindowEvent& rVclWindow break; case VclEventId::ToolboxHighlightOff: - ReleaseFocus_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + ReleaseFocus_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); break; case VclEventId::ToolboxItemAdded : - UpdateItem_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + UpdateItem_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); break; case VclEventId::ToolboxItemRemoved : @@ -566,8 +575,9 @@ void VCLXAccessibleToolBox::ProcessWindowEvent( const VclWindowEvent& rVclWindow case VclEventId::ToolboxItemWindowChanged: { - sal_Int32 nPos = (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()); + auto nPos = static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())); ToolBoxItemsMap::iterator aAccessiblePos( m_aAccessibleChildren.find( nPos ) ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! if ( m_aAccessibleChildren.end() != aAccessiblePos ) { implReleaseToolboxItem( aAccessiblePos, false ); @@ -576,17 +586,18 @@ void VCLXAccessibleToolBox::ProcessWindowEvent( const VclWindowEvent& rVclWindow Any aNewValue; aNewValue <<= getAccessibleChild(nPos); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! NotifyAccessibleEvent( AccessibleEventId::CHILD, Any(), aNewValue ); break; } case VclEventId::ToolboxItemTextChanged : - UpdateItemName_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + UpdateItemName_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); break; case VclEventId::ToolboxItemEnabled : case VclEventId::ToolboxItemDisabled : { - UpdateItemEnabled_Impl( (sal_Int32)reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData()) ); + UpdateItemEnabled_Impl( static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rVclWindowEvent.GetData())) ); break; } @@ -690,6 +701,7 @@ sal_Int32 SAL_CALL VCLXAccessibleToolBox::getAccessibleChildCount( ) VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) nCount = pToolBox->GetItemCount(); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! return nCount; } @@ -709,7 +721,7 @@ Reference< XAccessible > SAL_CALL VCLXAccessibleToolBox::getAccessibleChild( sal ToolBoxItemsMap::iterator aIter = m_aAccessibleChildren.find(i); if ( m_aAccessibleChildren.end() == aIter ) { - sal_uInt16 nItemId = pToolBox->GetItemId( (sal_uInt16)i ); + sal_uInt16 nItemId = pToolBox->GetItemId( i ); sal_uInt16 nHighlightItemId = pToolBox->GetHighlightItemId(); vcl::Window* pItemWindow = pToolBox->GetItemWindow( nItemId ); // not found -> create a new child @@ -749,9 +761,10 @@ Reference< XAccessible > SAL_CALL VCLXAccessibleToolBox::getAccessibleAtPoint( c VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pToolBox ) { - sal_uInt16 nItemPos = pToolBox->GetItemPos( VCLPoint( _rPoint ) ); - if ( nItemPos != TOOLBOX_ITEM_NOTFOUND ) + ToolBox::ImplToolItems::size_type nItemPos = pToolBox->GetItemPos( VCLPoint( _rPoint ) ); + if ( nItemPos != ToolBox::ITEM_NOTFOUND ) xAccessible = getAccessibleChild( nItemPos ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! } return xAccessible; @@ -764,13 +777,14 @@ Reference< XAccessible > VCLXAccessibleToolBox::GetItemWindowAccessible( const V VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); if ( pChildWindow && pToolBox ) { - sal_uInt16 nCount = pToolBox->GetItemCount(); - for (sal_uInt16 i = 0 ; i < nCount && !xReturn.is() ; ++i) + ToolBox::ImplToolItems::size_type nCount = pToolBox->GetItemCount(); + for (ToolBox::ImplToolItems::size_type i = 0 ; i < nCount && !xReturn.is() ; ++i) { sal_uInt16 nItemId = pToolBox->GetItemId( i ); vcl::Window* pItemWindow = pToolBox->GetItemWindow( nItemId ); if ( pItemWindow == pChildWindow ) xReturn = getAccessibleChild(i); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! } } return xReturn; @@ -792,8 +806,7 @@ void VCLXAccessibleToolBox::selectAccessibleChild( sal_Int32 nChildIndex ) if ( nChildIndex < 0 || nChildIndex >= getAccessibleChildCount() ) throw IndexOutOfBoundsException(); VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); - sal_uInt16 nPos = static_cast < sal_uInt16 > (nChildIndex); - pToolBox->ChangeHighlight( nPos ); + pToolBox->ChangeHighlight( nChildIndex ); } sal_Bool VCLXAccessibleToolBox::isAccessibleChildSelected( sal_Int32 nChildIndex ) @@ -802,8 +815,7 @@ sal_Bool VCLXAccessibleToolBox::isAccessibleChildSelected( sal_Int32 nChildIndex if ( nChildIndex < 0 || nChildIndex >= getAccessibleChildCount() ) throw IndexOutOfBoundsException(); VclPtr< ToolBox > pToolBox = GetAs< ToolBox >(); - sal_uInt16 nPos = static_cast < sal_uInt16 > (nChildIndex); - if ( pToolBox && pToolBox->GetHighlightItemId() == pToolBox->GetItemId( nPos ) ) + if ( pToolBox && pToolBox->GetHighlightItemId() == pToolBox->GetItemId( nChildIndex ) ) return true; else return false; diff --git a/accessibility/source/standard/vclxaccessibletoolboxitem.cxx b/accessibility/source/standard/vclxaccessibletoolboxitem.cxx index c2af5e1..bebc04f 100644 --- a/accessibility/source/standard/vclxaccessibletoolboxitem.cxx +++ b/accessibility/source/standard/vclxaccessibletoolboxitem.cxx @@ -74,11 +74,11 @@ VCLXAccessibleToolBoxItem::VCLXAccessibleToolBoxItem( ToolBox* _pToolBox, sal_In m_pExternalLock = static_cast< VCLExternalSolarLock* >( getExternalLock( ) ); OSL_ENSURE( m_pToolBox, "invalid toolbox" ); - m_nItemId = m_pToolBox->GetItemId( (sal_uInt16)m_nIndexInParent ); + m_nItemId = m_pToolBox->GetItemId( m_nIndexInParent ); m_sOldName = GetText(); m_bIsChecked = m_pToolBox->IsItemChecked( m_nItemId ); m_bIndeterminate = ( m_pToolBox->GetItemState( m_nItemId ) == TRISTATE_INDET ); - ToolBoxItemType eType = m_pToolBox->GetItemType( (sal_uInt16)m_nIndexInParent ); + ToolBoxItemType eType = m_pToolBox->GetItemType( m_nIndexInParent ); switch ( eType ) { case ToolBoxItemType::BUTTON : @@ -243,7 +243,7 @@ awt::Rectangle VCLXAccessibleToolBoxItem::implGetBounds( ) { awt::Rectangle aRect; if ( m_pToolBox ) - aRect = AWTRectangle( m_pToolBox->GetItemPosRect( (sal_uInt16)m_nIndexInParent ) ); + aRect = AWTRectangle( m_pToolBox->GetItemPosRect( m_nIndexInParent ) ); return aRect; } diff --git a/cui/source/customize/cfg.cxx b/cui/source/customize/cfg.cxx index e2067c3..0f47c18 100644 --- a/cui/source/customize/cfg.cxx +++ b/cui/source/customize/cfg.cxx @@ -5185,6 +5185,7 @@ SvxIconSelectorDialog::SvxIconSelectorDialog( vcl::Window *pWindow, pBtnDelete->SetClickHdl( LINK(this, SvxIconSelectorDialog, DeleteHdl) ); m_nNextId = pTbSymbol->GetItemCount()+1; + //TODO: ToolBox::ImplToolItems::size_type -> sal_uInt16! } SvxIconSelectorDialog::~SvxIconSelectorDialog() @@ -5196,9 +5197,9 @@ void SvxIconSelectorDialog::dispose() { if (pTbSymbol) { - sal_uInt16 nCount = pTbSymbol->GetItemCount(); + ToolBox::ImplToolItems::size_type nCount = pTbSymbol->GetItemCount(); - for (sal_uInt16 n = 0; n < nCount; ++n ) + for (ToolBox::ImplToolItems::size_type n = 0; n < nCount; ++n ) { sal_uInt16 nId = pTbSymbol->GetItemId(n); @@ -5222,7 +5223,7 @@ uno::Reference< graphic::XGraphic> SvxIconSelectorDialog::GetSelectedIcon() uno::Reference< graphic::XGraphic > result; sal_uInt16 nId; - for ( sal_uInt16 n = 0; n < pTbSymbol->GetItemCount(); ++n ) + for ( ToolBox::ImplToolItems::size_type n = 0; n < pTbSymbol->GetItemCount(); ++n ) { nId = pTbSymbol->GetItemId( n ); if ( pTbSymbol->IsItemChecked( nId ) ) @@ -5238,9 +5239,9 @@ IMPL_LINK( SvxIconSelectorDialog, SelectHdl, ToolBox *, pToolBox, void ) { (void)pToolBox; - sal_uInt16 nCount = pTbSymbol->GetItemCount(); + ToolBox::ImplToolItems::size_type nCount = pTbSymbol->GetItemCount(); - for (sal_uInt16 n = 0; n < nCount; ++n ) + for (ToolBox::ImplToolItems::size_type n = 0; n < nCount; ++n ) { sal_uInt16 nId = pTbSymbol->GetItemId( n ); @@ -5295,9 +5296,9 @@ IMPL_LINK_NOARG( SvxIconSelectorDialog, DeleteHdl, Button *, void ) OUString message = CUI_RES( RID_SVXSTR_DELETE_ICON_CONFIRM ); if (ScopedVclPtrInstance<WarningBox>(this, WinBits(WB_OK_CANCEL), message)->Execute() == RET_OK) { - sal_uInt16 nCount = pTbSymbol->GetItemCount(); + ToolBox::ImplToolItems::size_type nCount = pTbSymbol->GetItemCount(); - for (sal_uInt16 n = 0; n < nCount; ++n ) + for (ToolBox::ImplToolItems::size_type n = 0; n < nCount; ++n ) { sal_uInt16 nId = pTbSymbol->GetItemId( n ); @@ -5353,8 +5354,8 @@ bool SvxIconSelectorDialog::ReplaceGraphicItem( } bool bResult( false ); - sal_uInt16 nCount = pTbSymbol->GetItemCount(); - for (sal_uInt16 n = 0; n < nCount; ++n ) + ToolBox::ImplToolItems::size_type nCount = pTbSymbol->GetItemCount(); + for (ToolBox::ImplToolItems::size_type n = 0; n < nCount; ++n ) { sal_uInt16 nId = pTbSymbol->GetItemId( n ); diff --git a/extensions/source/bibliography/toolbar.cxx b/extensions/source/bibliography/toolbar.cxx index 1a0b71ea..fec960f 100644 --- a/extensions/source/bibliography/toolbar.cxx +++ b/extensions/source/bibliography/toolbar.cxx @@ -258,7 +258,7 @@ void BibToolBar::dispose() void BibToolBar::InitListener() { - sal_uInt16 nCount=GetItemCount(); + ToolBox::ImplToolItems::size_type nCount=GetItemCount(); uno::Reference< frame::XDispatch > xDisp(xController,UNO_QUERY); uno::Reference< util::XURLTransformer > xTrans( util::URLTransformer::create(comphelper::getProcessComponentContext()) ); @@ -270,7 +270,7 @@ void BibToolBar::InitListener() BibToolBarListener* pQuery=new BibTBQueryMenuListener(this, aQueryURL.Complete, nTBC_BT_AUTOFILTER); xDisp->addStatusListener(uno::Reference< frame::XStatusListener > (pQuery),aQueryURL); - for(sal_uInt16 nPos=0;nPos<nCount;nPos++) + for(ToolBox::ImplToolItems::size_type nPos=0;nPos<nCount;nPos++) { sal_uInt16 nId=GetItemId(nPos); if(!nId || nId== nTBC_FT_SOURCE || nId == nTBC_FT_QUERY) diff --git a/forms/source/solar/control/navtoolbar.cxx b/forms/source/solar/control/navtoolbar.cxx index f2be671..169ceb3 100644 --- a/forms/source/solar/control/navtoolbar.cxx +++ b/forms/source/solar/control/navtoolbar.cxx @@ -171,7 +171,7 @@ namespace frm pPositionWindow->setDispatcher( _pDispatcher ); // update feature states - for ( sal_uInt16 nPos = 0; nPos < m_pToolbar->GetItemCount(); ++nPos ) + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < m_pToolbar->GetItemCount(); ++nPos ) { sal_uInt16 nItemId = m_pToolbar->GetItemId( nPos ); @@ -199,7 +199,7 @@ namespace frm void NavigationToolBar::enableFeature( sal_Int16 _nFeatureId, bool _bEnabled ) { - DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != TOOLBOX_ITEM_NOTFOUND, + DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != ToolBox::ITEM_NOTFOUND, "NavigationToolBar::enableFeature: invalid id!" ); implEnableItem( (sal_uInt16)_nFeatureId, _bEnabled ); @@ -208,7 +208,7 @@ namespace frm void NavigationToolBar::checkFeature( sal_Int16 _nFeatureId, bool _bEnabled ) { - DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != TOOLBOX_ITEM_NOTFOUND, + DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != ToolBox::ITEM_NOTFOUND, "NavigationToolBar::checkFeature: invalid id!" ); m_pToolbar->CheckItem( (sal_uInt16)_nFeatureId, _bEnabled ); @@ -217,7 +217,7 @@ namespace frm void NavigationToolBar::setFeatureText( sal_Int16 _nFeatureId, const OUString& _rText ) { - DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != TOOLBOX_ITEM_NOTFOUND, + DBG_ASSERT( m_pToolbar->GetItemPos( (sal_uInt16)_nFeatureId ) != ToolBox::ITEM_NOTFOUND, "NavigationToolBar::checkFeature: invalid id!" ); vcl::Window* pItemWindow = m_pToolbar->GetItemWindow( (sal_uInt16)_nFeatureId ); @@ -343,14 +343,14 @@ namespace frm if ( !m_pImageProvider ) return; - const sal_uInt16 nItemCount = m_pToolbar->GetItemCount(); + const ToolBox::ImplToolItems::size_type nItemCount = m_pToolbar->GetItemCount(); // collect the FormFeatures in the toolbar typedef ::std::vector< sal_Int16 > FormFeatures; FormFeatures aFormFeatures; aFormFeatures.reserve( nItemCount ); - for ( sal_uInt16 i=0; i<nItemCount; ++i ) + for ( ToolBox::ImplToolItems::size_type i=0; i<nItemCount; ++i ) { sal_uInt16 nId = m_pToolbar->GetItemId( i ); if ( ( ToolBoxItemType::BUTTON == m_pToolbar->GetItemType( i ) ) && !isArtificialItem( nId ) ) @@ -552,7 +552,7 @@ namespace frm void NavigationToolBar::forEachItemWindow( ItemWindowHandler _handler ) { - for ( sal_uInt16 item = 0; item < m_pToolbar->GetItemCount(); ++item ) + for ( ToolBox::ImplToolItems::size_type item = 0; item < m_pToolbar->GetItemCount(); ++item ) { sal_uInt16 nItemId = m_pToolbar->GetItemId( item ); vcl::Window* pItemWindow = m_pToolbar->GetItemWindow( nItemId ); @@ -563,7 +563,7 @@ namespace frm void NavigationToolBar::forEachItemWindow( ItemWindowHandler2 _handler, const void* _pParam ) { - for ( sal_uInt16 item = 0; item < m_pToolbar->GetItemCount(); ++item ) + for ( ToolBox::ImplToolItems::size_type item = 0; item < m_pToolbar->GetItemCount(); ++item ) { sal_uInt16 nItemId = m_pToolbar->GetItemId( item ); vcl::Window* pItemWindow = m_pToolbar->GetItemWindow( nItemId ); diff --git a/framework/inc/uielement/toolbarmerger.hxx b/framework/inc/uielement/toolbarmerger.hxx index 683af86..665f37b 100644 --- a/framework/inc/uielement/toolbarmerger.hxx +++ b/framework/inc/uielement/toolbarmerger.hxx @@ -63,7 +63,7 @@ typedef ::std::vector< AddonToolbarItem > AddonToolbarItemContainer; struct ReferenceToolbarPathInfo { VclPtr<ToolBox> pToolbar; - sal_uInt16 nPos; + ToolBox::ImplToolItems::size_type nPos; bool bResult; }; @@ -88,7 +88,7 @@ class ToolBarMerger const OUString& rReferencePoint ); static bool ProcessMergeOperation( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, @@ -97,7 +97,7 @@ class ToolBarMerger const AddonToolbarItemContainer& rItems ); static bool ProcessMergeFallback( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, @@ -106,7 +106,7 @@ class ToolBarMerger const AddonToolbarItemContainer& rItems ); static bool MergeItems( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16 nModIndex, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, @@ -114,14 +114,14 @@ class ToolBarMerger const AddonToolbarItemContainer& rAddonToolbarItems ); static bool ReplaceItem( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, const AddonToolbarItemContainer& rAddonToolbarItems ); static bool RemoveItems( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, const OUString& rMergeCommandParameter ); static ::cppu::OWeakObject* CreateController( @@ -134,7 +134,7 @@ class ToolBarMerger const OUString& rControlType ); static void CreateToolbarItem( ToolBox* pToolbox, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16 nItemId, const AddonToolbarItem& rAddonToolbarItem ); diff --git a/framework/source/uielement/addonstoolbarmanager.cxx b/framework/source/uielement/addonstoolbarmanager.cxx index 6257c12..24e98b0 100644 --- a/framework/source/uielement/addonstoolbarmanager.cxx +++ b/framework/source/uielement/addonstoolbarmanager.cxx @@ -142,7 +142,7 @@ void SAL_CALL AddonsToolBarManager::dispose() { // Remove addon specific data from toolbar items. SolarMutexGuard g; - for ( sal_uInt16 n = 0; n < m_pToolBar->GetItemCount(); n++ ) + for ( ToolBox::ImplToolItems::size_type n = 0; n < m_pToolBar->GetItemCount(); n++ ) { sal_uInt16 nId( m_pToolBar->GetItemId( n ) ); @@ -171,7 +171,7 @@ bool AddonsToolBarManager::MenuItemAllowed( sal_uInt16 nId ) const void AddonsToolBarManager::RefreshImages() { bool bBigImages( SvtMiscOptions().AreCurrentSymbolsLarge() ); - for ( sal_uInt16 nPos = 0; nPos < m_pToolBar->GetItemCount(); nPos++ ) + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < m_pToolBar->GetItemCount(); nPos++ ) { sal_uInt16 nId( m_pToolBar->GetItemId( nPos ) ); @@ -239,7 +239,7 @@ void AddonsToolBarManager::FillToolbar( const Sequence< Sequence< PropertyValue { if ( aURL == "private:separator" ) // toolbox item separator { - sal_uInt16 nCount = m_pToolBar->GetItemCount(); + ToolBox::ImplToolItems::size_type nCount = m_pToolBar->GetItemCount(); if ( nCount > 0 && ( m_pToolBar->GetItemType( nCount-1 ) != ToolBoxItemType::SEPARATOR ) && nElements > 0 ) { nElements = 0; @@ -248,7 +248,7 @@ void AddonsToolBarManager::FillToolbar( const Sequence< Sequence< PropertyValue } else { - sal_uInt16 nCount = m_pToolBar->GetItemCount(); + ToolBox::ImplToolItems::size_type nCount = m_pToolBar->GetItemCount(); if ( bAppendSeparator && nCount > 0 && ( m_pToolBar->GetItemType( nCount-1 ) != ToolBoxItemType::SEPARATOR )) { // We have to append a separator first if the last item is not a separator @@ -451,7 +451,7 @@ IMPL_LINK( AddonsToolBarManager, DataChanged, DataChangedEvent const *, pDataCha CheckAndUpdateImages(); } - for ( sal_uInt16 nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) { const sal_uInt16 nId = m_pToolBar->GetItemId(nPos); vcl::Window* pWindow = m_pToolBar->GetItemWindow( nId ); diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx index d72a165..0ad9f1a 100644 --- a/framework/source/uielement/toolbarmanager.cxx +++ b/framework/source/uielement/toolbarmanager.cxx @@ -223,7 +223,7 @@ void ToolBarManager::Destroy() } // Delete the additional add-ons data - for ( sal_uInt16 i = 0; i < m_pToolBar->GetItemCount(); i++ ) + for ( ToolBox::ImplToolItems::size_type i = 0; i < m_pToolBar->GetItemCount(); i++ ) { sal_uInt16 nItemId = m_pToolBar->GetItemId( i ); if ( nItemId > 0 ) @@ -639,7 +639,7 @@ void ToolBarManager::RemoveControllers() // destroyed by the dispose() at the XComponent. This is needed // as VCL code later tries to access the item window data in certain // dtors where the item window is already invalid! - for ( sal_uInt16 i = 0; i < m_pToolBar->GetItemCount(); i++ ) + for ( ToolBox::ImplToolItems::size_type i = 0; i < m_pToolBar->GetItemCount(); i++ ) { sal_uInt16 nItemId = m_pToolBar->GetItemId( i ); if ( nItemId > 0 ) @@ -669,7 +669,7 @@ void ToolBarManager::CreateControllers() bool bHasDisabledEntries = SvtCommandOptions().HasEntries( SvtCommandOptions::CMDOPTION_DISABLED ); SvtCommandOptions aCmdOptions; - for ( sal_uInt16 i = 0; i < m_pToolBar->GetItemCount(); i++ ) + for ( ToolBox::ImplToolItems::size_type i = 0; i < m_pToolBar->GetItemCount(); i++ ) { sal_uInt16 nId = m_pToolBar->GetItemId( i ); if ( nId == 0 ) @@ -1177,7 +1177,7 @@ void ToolBarManager::FillOverflowToolbar( ToolBox* pParent ) { CommandInfo aCmdInfo; bool bInsertSeparator = false; - for ( sal_uInt16 i = 0; i < pParent->GetItemCount(); ++i ) + for ( ToolBox::ImplToolItems::size_type i = 0; i < pParent->GetItemCount(); ++i ) { sal_uInt16 nId = pParent->GetItemId( i ); if ( pParent->IsItemClipped( nId ) ) @@ -1422,7 +1422,6 @@ void ToolBarManager::AddCustomizeMenuItems(ToolBox* pToolBar) if ( m_pToolBar->IsCustomize() ) { - sal_uInt16 nPos( 0 ); ::PopupMenu* pVisibleItemsPopupMenu( aQuickCustomizationMenu->GetPopupMenu( 1 )); bool bIsFloating( false ); @@ -1459,18 +1458,22 @@ void ToolBarManager::AddCustomizeMenuItems(ToolBox* pToolBar) std::map< OUString, Image > commandToImage; // Go through all toolbar items and add them to the context menu - for ( nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) { if ( m_pToolBar->GetItemType(nPos) == ToolBoxItemType::BUTTON ) { sal_uInt16 nId = m_pToolBar->GetItemId(nPos); OUString aCommandURL = m_pToolBar->GetItemCommand( nId ); pVisibleItemsPopupMenu->InsertItem( STARTID_CUSTOMIZE_POPUPMENU+nPos, m_pToolBar->GetItemText( nId ), MenuItemBits::CHECKABLE ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_uInt16! pVisibleItemsPopupMenu->CheckItem( STARTID_CUSTOMIZE_POPUPMENU+nPos, m_pToolBar->IsItemVisible( nId ) ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_uInt16! pVisibleItemsPopupMenu->SetItemCommand( STARTID_CUSTOMIZE_POPUPMENU+nPos, aCommandURL ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_uInt16! Image aImage(vcl::CommandInfoProvider::GetImageForCommand(aCommandURL, m_xFrame)); commandToImage[aCommandURL] = aImage; pVisibleItemsPopupMenu->SetItemImage( STARTID_CUSTOMIZE_POPUPMENU+nPos, aImage ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_uInt16! } else { @@ -1480,7 +1483,7 @@ void ToolBarManager::AddCustomizeMenuItems(ToolBox* pToolBar) // Now we go through all the contextual menu to update the icons std::map< OUString, Image >::iterator it; - for ( nPos = 0; nPos < pMenu->GetItemCount(); ++nPos ) + for ( sal_uInt16 nPos = 0; nPos < pMenu->GetItemCount(); ++nPos ) { sal_uInt16 nId = pMenu->GetItemId( nPos ); OUString cmdUrl = pMenu->GetItemCommand( nId ); @@ -1784,7 +1787,7 @@ IMPL_LINK( ToolBarManager, DataChanged, DataChangedEvent const *, pDataChangedEv CheckAndUpdateImages(); } - for ( sal_uInt16 nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < m_pToolBar->GetItemCount(); ++nPos ) { const sal_uInt16 nId = m_pToolBar->GetItemId(nPos); vcl::Window* pWindow = m_pToolBar->GetItemWindow( nId ); diff --git a/framework/source/uielement/toolbarmerger.cxx b/framework/source/uielement/toolbarmerger.cxx index 5c89dfa..2973b2c 100644 --- a/framework/source/uielement/toolbarmerger.cxx +++ b/framework/source/uielement/toolbarmerger.cxx @@ -238,11 +238,11 @@ ReferenceToolbarPathInfo ToolBarMerger::FindReferencePoint( ReferenceToolbarPathInfo aResult; aResult.bResult = false; aResult.pToolbar = pToolbar; - aResult.nPos = TOOLBOX_ITEM_NOTFOUND; + aResult.nPos = ToolBox::ITEM_NOTFOUND; - const sal_uInt16 nSize( pToolbar->GetItemCount() ); + const ToolBox::ImplToolItems::size_type nSize( pToolbar->GetItemCount() ); - for ( sal_uInt16 i = 0; i < nSize; i++ ) + for ( ToolBox::ImplToolItems::size_type i = 0; i < nSize; i++ ) { const sal_uInt16 nItemId = pToolbar->GetItemId( i ); if ( nItemId > 0 ) @@ -308,7 +308,7 @@ ReferenceToolbarPathInfo ToolBarMerger::FindReferencePoint( */ bool ToolBarMerger::ProcessMergeOperation( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, @@ -371,7 +371,7 @@ bool ToolBarMerger::ProcessMergeOperation( */ bool ToolBarMerger::ProcessMergeFallback( ToolBox* pToolbar, - sal_uInt16 /*nPos*/, + ToolBox::ImplToolItems::size_type /*nPos*/, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, @@ -391,7 +391,7 @@ bool ToolBarMerger::ProcessMergeFallback( if ( rMergeFallback == MERGEFALLBACK_ADDFIRST ) return MergeItems( pToolbar, 0, 0, rItemId, rCommandMap, rModuleIdentifier, rItems ); else if ( rMergeFallback == MERGEFALLBACK_ADDLAST ) - return MergeItems( pToolbar, TOOLBOX_APPEND, 0, rItemId, rCommandMap, rModuleIdentifier, rItems ); + return MergeItems( pToolbar, ToolBox::APPEND, 0, rItemId, rCommandMap, rModuleIdentifier, rItems ); } return false; @@ -435,7 +435,7 @@ bool ToolBarMerger::ProcessMergeFallback( */ bool ToolBarMerger::MergeItems( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16 nModIndex, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, @@ -450,12 +450,16 @@ bool ToolBarMerger::MergeItems( const AddonToolbarItem& rItem = rAddonToolbarItems[i]; if ( IsCorrectContext( rItem.aContext, rModuleIdentifier )) { - sal_Int32 nInsPos = nPos+nModIndex+i; - if ( nInsPos > sal_Int32( pToolbar->GetItemCount() )) - nInsPos = TOOLBOX_APPEND; + ToolBox::ImplToolItems::size_type nInsPos = nPos; + if (nInsPos != ToolBox::APPEND) + { + nInsPos += nModIndex+i; + if ( nInsPos > pToolbar->GetItemCount() ) + nInsPos = ToolBox::APPEND; + } if ( rItem.aCommandURL == TOOLBOXITEM_SEPARATOR_STR ) - pToolbar->InsertSeparator( sal_uInt16( nInsPos )); + pToolbar->InsertSeparator( nInsPos ); else { CommandToInfoMap::iterator pIter = rCommandMap.find( rItem.aCommandURL ); @@ -471,7 +475,7 @@ bool ToolBarMerger::MergeItems( pIter->second.aIds.push_back( rItemId ); } - ToolBarMerger::CreateToolbarItem( pToolbar, sal_uInt16( nInsPos ), rItemId, rItem ); + ToolBarMerger::CreateToolbarItem( pToolbar, nInsPos, rItemId, rItem ); } ++nIndex; @@ -521,7 +525,7 @@ bool ToolBarMerger::MergeItems( */ bool ToolBarMerger::ReplaceItem( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, sal_uInt16& rItemId, CommandToInfoMap& rCommandMap, const OUString& rModuleIdentifier, @@ -558,7 +562,7 @@ bool ToolBarMerger::ReplaceItem( */ bool ToolBarMerger::RemoveItems( ToolBox* pToolbar, - sal_uInt16 nPos, + ToolBox::ImplToolItems::size_type nPos, const OUString& rMergeCommandParameter ) { sal_Int32 nCount = rMergeCommandParameter.toInt32(); @@ -633,7 +637,7 @@ bool ToolBarMerger::RemoveItems( return pResult; } -void ToolBarMerger::CreateToolbarItem( ToolBox* pToolbar, sal_uInt16 nPos, sal_uInt16 nItemId, const AddonToolbarItem& rItem ) +void ToolBarMerger::CreateToolbarItem( ToolBox* pToolbar, ToolBox::ImplToolItems::size_type nPos, sal_uInt16 nItemId, const AddonToolbarItem& rItem ) { pToolbar->InsertItem( nItemId, rItem.aLabel, ToolBoxItemBits::NONE, nPos ); pToolbar->SetItemCommand( nItemId, rItem.aCommandURL ); diff --git a/include/sfx2/sidebar/SidebarToolBox.hxx b/include/sfx2/sidebar/SidebarToolBox.hxx index 793d2a7..1dc2963 100644 --- a/include/sfx2/sidebar/SidebarToolBox.hxx +++ b/include/sfx2/sidebar/SidebarToolBox.hxx @@ -48,7 +48,7 @@ public: const css::uno::Reference<css::frame::XFrame>& rFrame, ToolBoxItemBits nBits, const Size& rRequestedSize, - sal_uInt16 nPos = TOOLBOX_APPEND) override; + ImplToolItems::size_type nPos = APPEND) override; virtual bool EventNotify(NotifyEvent& rEvent) override; diff --git a/include/vcl/toolbox.hxx b/include/vcl/toolbox.hxx index d9583e8..095cc68 100644 --- a/include/vcl/toolbox.hxx +++ b/include/vcl/toolbox.hxx @@ -26,6 +26,8 @@ #include <vcl/dockwin.hxx> #include <vcl/image.hxx> #include <o3tl/typed_flags_set.hxx> + +#include <limits> #include <vector> #include <com/sun/star/frame/XFrame.hpp> @@ -41,9 +43,6 @@ class PopupMenu; #define TOOLBOX_STYLE_FLAT ((sal_uInt16)0x0004) -#define TOOLBOX_APPEND ((sal_uInt16)0xFFFF) -#define TOOLBOX_ITEM_NOTFOUND ((sal_uInt16)0xFFFF) - // item ids in the custom menu may not exceed this constant #define TOOLBOX_MENUITEM_START ((sal_uInt16)0x1000) @@ -90,6 +89,15 @@ class VCL_DLLPUBLIC ToolBox : public DockingWindow friend class FloatingWindow; friend class ImplTBDragMgr; +public: + using ImplToolItems = std::vector<ImplToolItem>; + + static constexpr auto APPEND + = std::numeric_limits<ImplToolItems::size_type>::max(); + + static constexpr auto ITEM_NOTFOUND + = std::numeric_limits<ImplToolItems::size_type>::max(); + private: ImplToolBoxPrivateData* mpData; std::vector<ImplToolSize> maFloatSizes; @@ -118,7 +126,7 @@ private: sal_uInt16 mnHighItemId; sal_uInt16 mnCurItemId; sal_uInt16 mnDownItemId; - sal_uInt16 mnCurPos; + ImplToolItems::size_type mnCurPos; sal_uInt16 mnLines; // total number of toolbox lines sal_uInt16 mnCurLine; // the currently visible line sal_uInt16 mnCurLines; // number of lines due to line breaking @@ -168,7 +176,7 @@ private: public: using Window::ImplInit; private: - SAL_DLLPRIVATE void InvalidateItem(sal_uInt16 nPosition); + SAL_DLLPRIVATE void InvalidateItem(ImplToolItems::size_type nPosition); SAL_DLLPRIVATE void InvalidateSpin(bool bInvalidateUpper = true, bool bInvalidateLower = true); SAL_DLLPRIVATE void InvalidateMenuButton(); @@ -182,11 +190,11 @@ private: SAL_DLLPRIVATE sal_uInt16 ImplCalcBreaks( long nWidth, long* pMaxLineWidth, bool bCalcHorz ); SAL_DLLPRIVATE void ImplFormat( bool bResize = false ); SAL_DLLPRIVATE void ImplDrawSpin(vcl::RenderContext& rRenderContext); - SAL_DLLPRIVATE void ImplDrawSeparator(vcl::RenderContext& rRenderContext, sal_uInt16 nPos, const Rectangle& rRect); - SAL_DLLPRIVATE void ImplDrawItem(vcl::RenderContext& rRenderContext, sal_uInt16 nPos, sal_uInt16 nHighlight ); + SAL_DLLPRIVATE void ImplDrawSeparator(vcl::RenderContext& rRenderContext, ImplToolItems::size_type nPos, const Rectangle& rRect); + SAL_DLLPRIVATE void ImplDrawItem(vcl::RenderContext& rRenderContext, ImplToolItems::size_type nPos, sal_uInt16 nHighlight ); using Window::ImplInvalidate; SAL_DLLPRIVATE void ImplInvalidate( bool bNewCalc = false, bool bFullPaint = false ); - SAL_DLLPRIVATE void ImplUpdateItem( sal_uInt16 nIndex = 0xFFFF ); + SAL_DLLPRIVATE void ImplUpdateItem( ImplToolItems::size_type nIndex = ITEM_NOTFOUND ); SAL_DLLPRIVATE bool ImplHandleMouseMove( const MouseEvent& rMEvt, bool bRepeat = false ); SAL_DLLPRIVATE bool ImplHandleMouseButtonUp( const MouseEvent& rMEvt, bool bCancel = false ); SAL_DLLPRIVATE void ImplChangeHighlight( ImplToolItem* pItem, bool bNoGrabFocus = false ); @@ -250,7 +258,7 @@ public: static SAL_DLLPRIVATE sal_uInt16 ImplTestLineSize( ToolBox* pThis, const Point& rPos ); static SAL_DLLPRIVATE void ImplLineSizing( ToolBox* pThis, const Point& rPos, Rectangle& rRect, sal_uInt16 nLineMode ); static SAL_DLLPRIVATE sal_uInt16 ImplFindItemPos( ToolBox* pBox, const Point& rPos ); - static SAL_DLLPRIVATE std::vector<ImplToolItem>::size_type ImplFindItemPos( const ImplToolItem* pItem, const std::vector< ImplToolItem >& rList ); + static SAL_DLLPRIVATE ImplToolItems::size_type ImplFindItemPos( const ImplToolItem* pItem, const ImplToolItems& rList ); SAL_DLLPRIVATE void ImplDrawMenuButton(vcl::RenderContext& rRenderContext, bool bHighlight); SAL_DLLPRIVATE void ImplDrawButton(vcl::RenderContext& rRenderContext, const Rectangle &rRect, sal_uInt16 highlight, bool bChecked, bool bEnabled, bool bIsWindow); static SAL_DLLPRIVATE sal_uInt16 ImplCountLineBreaks( const ToolBox *pThis ); @@ -305,24 +313,24 @@ public: const css::uno::Reference<css::frame::XFrame>& rFrame, ToolBoxItemBits nBits, const Size& rRequestedSize, - sal_uInt16 nPos = TOOLBOX_APPEND ); + ImplToolItems::size_type nPos = APPEND ); void InsertItem( sal_uInt16 nItemId, const Image& rImage, ToolBoxItemBits nBits = ToolBoxItemBits::NONE, - sal_uInt16 nPos = TOOLBOX_APPEND ); + ImplToolItems::size_type nPos = APPEND ); void InsertItem( sal_uInt16 nItemId, const Image& rImage, const OUString& rText, ToolBoxItemBits nBits = ToolBoxItemBits::NONE, - sal_uInt16 nPos = TOOLBOX_APPEND ); + ImplToolItems::size_type nPos = APPEND ); void InsertItem( sal_uInt16 nItemId, const OUString& rText, ToolBoxItemBits nBits = ToolBoxItemBits::NONE, - sal_uInt16 nPos = TOOLBOX_APPEND ); + ImplToolItems::size_type nPos = APPEND ); void InsertWindow( sal_uInt16 nItemId, vcl::Window* pWindow, ToolBoxItemBits nBits = ToolBoxItemBits::NONE, - sal_uInt16 nPos = TOOLBOX_APPEND ); + ImplToolItems::size_type nPos = APPEND ); void InsertSpace(); - void InsertSeparator( sal_uInt16 nPos = TOOLBOX_APPEND, sal_uInt16 nPixSize = 0 ); - void InsertBreak( sal_uInt16 nPos = TOOLBOX_APPEND ); - void RemoveItem( sal_uInt16 nPos ); + void InsertSeparator( ImplToolItems::size_type nPos = APPEND, sal_uInt16 nPixSize = 0 ); + void InsertBreak( ImplToolItems::size_type nPos = APPEND ); + void RemoveItem( ImplToolItems::size_type nPos ); void CopyItem( const ToolBox& rToolBox, sal_uInt16 nItemId ); void Clear(); @@ -343,16 +351,16 @@ public: // Used to enable/disable scrolling one page at a time for toolbar void SetPageScroll( bool b ); - sal_uInt16 GetItemCount() const; - ToolBoxItemType GetItemType( sal_uInt16 nPos ) const; - sal_uInt16 GetItemPos( sal_uInt16 nItemId ) const; - sal_uInt16 GetItemPos( const Point& rPos ) const; - sal_uInt16 GetItemId( sal_uInt16 nPos ) const; + ImplToolItems::size_type GetItemCount() const; + ToolBoxItemType GetItemType( ImplToolItems::size_type nPos ) const; + ImplToolItems::size_type GetItemPos( sal_uInt16 nItemId ) const; + ImplToolItems::size_type GetItemPos( const Point& rPos ) const; + sal_uInt16 GetItemId( ImplToolItems::size_type nPos ) const; sal_uInt16 GetItemId( const Point& rPos ) const; /// Map the command name (like .uno:Save) back to item id. sal_uInt16 GetItemId( const OUString& rCommand ) const; Rectangle GetItemRect( sal_uInt16 nItemId ) const; - Rectangle GetItemPosRect( sal_uInt16 nPos ) const; + Rectangle GetItemPosRect( ImplToolItems::size_type nPos ) const; Rectangle GetOverflowRect() const; /// Returns size of the bitmap / text that is inside this toolbox item. @@ -506,7 +514,7 @@ public: static Size GetDefaultImageSize(ToolBoxButtonSize eToolBoxButtonSize); Size GetDefaultImageSize() const; - void ChangeHighlight( sal_uInt16 nPos ); + void ChangeHighlight( ImplToolItems::size_type nPos ); void SetToolbarLayoutMode( ToolBoxLayoutMode eLayout ); void statusChanged(const css::frame::FeatureStateEvent& rEvent); diff --git a/reportdesign/source/ui/dlg/AddField.cxx b/reportdesign/source/ui/dlg/AddField.cxx index ad8fdde..09e3726 100644 --- a/reportdesign/source/ui/dlg/AddField.cxx +++ b/reportdesign/source/ui/dlg/AddField.cxx @@ -316,8 +316,8 @@ void OAddFieldWindow::Update() { // ListBox loeschen m_pListBox->Clear(); - const sal_uInt16 nItemCount = m_aActions->GetItemCount(); - for (sal_uInt16 j = 0; j< nItemCount; ++j) + const ToolBox::ImplToolItems::size_type nItemCount = m_aActions->GetItemCount(); + for (ToolBox::ImplToolItems::size_type j = 0; j< nItemCount; ++j) { m_aActions->EnableItem(m_aActions->GetItemId(j),false); } @@ -363,7 +363,7 @@ void OAddFieldWindow::Update() SetText( aTitle ); if ( !m_aCommandName.isEmpty() ) { - for (sal_uInt16 i = 0; i < nItemCount; ++i) + for (ToolBox::ImplToolItems::size_type i = 0; i < nItemCount; ++i) { m_aActions->EnableItem(m_aActions->GetItemId(i)); } @@ -473,8 +473,8 @@ IMPL_LINK_NOARG( OAddFieldWindow, OnSortAction, ToolBox*, void ) { if (nCurItem == m_nRemoveSortId || !m_aActions->IsItemChecked(nCurItem)) { - const sal_uInt16 nItemCount = m_aActions->GetItemCount(); - for (sal_uInt16 j = 0; j< nItemCount; ++j) + const ToolBox::ImplToolItems::size_type nItemCount = m_aActions->GetItemCount(); + for (ToolBox::ImplToolItems::size_type j = 0; j< nItemCount; ++j) { const sal_uInt16 nItemId = m_aActions->GetItemId(j); if ( nCurItem != nItemId ) diff --git a/reportdesign/source/ui/dlg/Condition.cxx b/reportdesign/source/ui/dlg/Condition.cxx index 6136ef5..f93326f 100644 --- a/reportdesign/source/ui/dlg/Condition.cxx +++ b/reportdesign/source/ui/dlg/Condition.cxx @@ -392,8 +392,8 @@ void Condition::updateToolbar(const uno::Reference< report::XReportControlFormat OSL_ENSURE(_xReportControlFormat.is(),"XReportControlFormat is NULL!"); if ( _xReportControlFormat.is() ) { - sal_uInt16 nItemCount = m_pActions->GetItemCount(); - for (sal_uInt16 j = 0; j< nItemCount; ++j) + ToolBox::ImplToolItems::size_type nItemCount = m_pActions->GetItemCount(); + for (ToolBox::ImplToolItems::size_type j = 0; j< nItemCount; ++j) { sal_uInt16 nItemId = m_pActions->GetItemId(j); m_pActions->CheckItem( nItemId, OReportController::isFormatCommandEnabled(mapToolbarItemToSlotId(nItemId), diff --git a/reportdesign/source/ui/misc/toolboxcontroller.cxx b/reportdesign/source/ui/misc/toolboxcontroller.cxx index b482c78..6db2ee4 100644 --- a/reportdesign/source/ui/misc/toolboxcontroller.cxx +++ b/reportdesign/source/ui/misc/toolboxcontroller.cxx @@ -123,8 +123,8 @@ void SAL_CALL OToolboxController::initialize( const Sequence< Any >& _rArguments VclPtr< ToolBox > pToolBox = static_cast<ToolBox*>(VCLUnoHelper::GetWindow(getParent()).get()); if ( pToolBox ) { - const sal_uInt16 nCount = pToolBox->GetItemCount(); - for (sal_uInt16 nPos = 0; nPos < nCount; ++nPos) + const ToolBox::ImplToolItems::size_type nCount = pToolBox->GetItemCount(); + for (ToolBox::ImplToolItems::size_type nPos = 0; nPos < nCount; ++nPos) { const sal_uInt16 nItemId = pToolBox->GetItemId(nPos); if ( pToolBox->GetItemCommand(nItemId) == m_aCommandURL ) diff --git a/sfx2/source/sidebar/SidebarToolBox.cxx b/sfx2/source/sidebar/SidebarToolBox.cxx index 5a52fe8..4874694 100644 --- a/sfx2/source/sidebar/SidebarToolBox.cxx +++ b/sfx2/source/sidebar/SidebarToolBox.cxx @@ -109,7 +109,7 @@ ToolBoxButtonSize SidebarToolBox::GetDefaultButtonSize() const void SidebarToolBox::InsertItem(const OUString& rCommand, const css::uno::Reference<css::frame::XFrame>& rFrame, - ToolBoxItemBits nBits, const Size& rRequestedSize, sal_uInt16 nPos) + ToolBoxItemBits nBits, const Size& rRequestedSize, ImplToolItems::size_type nPos) { OUString aCommand( rCommand ); diff --git a/sfx2/source/view/viewsh.cxx b/sfx2/source/view/viewsh.cxx index 9053ebe..59a40a5 100644 --- a/sfx2/source/view/viewsh.cxx +++ b/sfx2/source/view/viewsh.cxx @@ -443,8 +443,8 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq ) ToolBox* pTextToolbox = dynamic_cast< ToolBox* >( pWin.get() ); if( pTextToolbox ) { - sal_uInt16 nItemCount = pTextToolbox->GetItemCount(); - for( sal_uInt16 nItem = 0; nItem < nItemCount; ++nItem ) + ToolBox::ImplToolItems::size_type nItemCount = pTextToolbox->GetItemCount(); + for( ToolBox::ImplToolItems::size_type nItem = 0; nItem < nItemCount; ++nItem ) { sal_uInt16 nItemId = pTextToolbox->GetItemId( nItem ); const OUString& rCommand = pTextToolbox->GetItemCommand( nItemId ); diff --git a/svtools/source/uno/toolboxcontroller.cxx b/svtools/source/uno/toolboxcontroller.cxx index 4aa9032..32ce1a7 100644 --- a/svtools/source/uno/toolboxcontroller.cxx +++ b/svtools/source/uno/toolboxcontroller.cxx @@ -765,8 +765,8 @@ bool ToolboxController::getToolboxId( sal_uInt16& rItemId, ToolBox** ppToolBox ) if( (m_nToolBoxId == SAL_MAX_UINT16) && pToolBox ) { - const sal_uInt16 nCount = pToolBox->GetItemCount(); - for ( sal_uInt16 nPos = 0; nPos < nCount; ++nPos ) + const ToolBox::ImplToolItems::size_type nCount = pToolBox->GetItemCount(); + for ( ToolBox::ImplToolItems::size_type nPos = 0; nPos < nCount; ++nPos ) { const sal_uInt16 nItemId = pToolBox->GetItemId( nPos ); if ( pToolBox->GetItemCommand( nItemId ) == m_aCommandURL ) diff --git a/svx/source/dialog/srchdlg.cxx b/svx/source/dialog/srchdlg.cxx index c990206..2929c37 100644 --- a/svx/source/dialog/srchdlg.cxx +++ b/svx/source/dialog/srchdlg.cxx @@ -2395,7 +2395,7 @@ static vcl::Window* lcl_GetSearchLabelWindow() css::uno::Reference< css::awt::XWindow > xWindow( xUIElement->getRealInterface(), css::uno::UNO_QUERY_THROW); VclPtr< ToolBox > pToolBox = static_cast<ToolBox*>( VCLUnoHelper::GetWindow(xWindow).get() ); - for (size_t i = 0; pToolBox && i < pToolBox->GetItemCount(); ++i) + for (ToolBox::ImplToolItems::size_type i = 0; pToolBox && i < pToolBox->GetItemCount(); ++i) { sal_uInt16 id = pToolBox->GetItemId(i); if (pToolBox->GetItemCommand(id) == ".uno:SearchLabel") diff --git a/svx/source/tbxctrls/tbunosearchcontrollers.cxx b/svx/source/tbxctrls/tbunosearchcontrollers.cxx index 430fa8c..9439189 100644 --- a/svx/source/tbxctrls/tbunosearchcontrollers.cxx +++ b/svx/source/tbxctrls/tbunosearchcontrollers.cxx @@ -89,8 +89,8 @@ void impl_executeSearch( const css::uno::Reference< css::uno::XComponentContext bool bSearchFormatted = false; if ( pToolBox ) { - sal_uInt16 nItemCount = pToolBox->GetItemCount(); - for ( sal_uInt16 i=0; i<nItemCount; ++i ) + ToolBox::ImplToolItems::size_type nItemCount = pToolBox->GetItemCount(); + for ( ToolBox::ImplToolItems::size_type i=0; i<nItemCount; ++i ) { sal_uInt16 id = pToolBox->GetItemId(i); OUString sItemCommand = pToolBox->GetItemCommand(id); @@ -526,8 +526,8 @@ void SAL_CALL FindTextToolbarController::initialize( const css::uno::Sequence< c ToolBox* pToolBox = static_cast<ToolBox*>(pWindow.get()); if ( pToolBox ) { - sal_uInt16 nItemCount = pToolBox->GetItemCount(); - for ( sal_uInt16 i=0; i<nItemCount; ++i ) + ToolBox::ImplToolItems::size_type nItemCount = pToolBox->GetItemCount(); + for ( ToolBox::ImplToolItems::size_type i=0; i<nItemCount; ++i ) { OUString sItemCommand = pToolBox->GetItemCommand(i); sal_uInt16 id = pToolBox->GetItemId(i); @@ -1435,8 +1435,8 @@ void SAL_CALL FindbarDispatcher::dispatch( const css::util::URL& aURL, const css ToolBox* pToolBox = static_cast<ToolBox*>(pWindow.get()); if ( pToolBox ) { - sal_uInt16 nItemCount = pToolBox->GetItemCount(); - for ( sal_uInt16 i=0; i<nItemCount; ++i ) + ToolBox::ImplToolItems::size_type nItemCount = pToolBox->GetItemCount(); + for ( ToolBox::ImplToolItems::size_type i=0; i<nItemCount; ++i ) { sal_uInt16 id = pToolBox->GetItemId(i); OUString sItemCommand = pToolBox->GetItemCommand(id); diff --git a/sw/source/uibase/ribbar/workctrl.cxx b/sw/source/uibase/ribbar/workctrl.cxx index cbc7aa9d..da38493 100644 --- a/sw/source/uibase/ribbar/workctrl.cxx +++ b/sw/source/uibase/ribbar/workctrl.cxx @@ -310,7 +310,7 @@ IMPL_LINK(SwScrollNaviPopup, SelectHdl, ToolBox*, pSet, void) m_pToolBox->SetItemText(NID_PREV, sQuickHelp[nSet - NID_START + NID_COUNT]); m_pInfoField->SetText(m_pToolBox->GetItemText(nSet)); // check the current button only - for(sal_uInt16 i = 0; i < NID_COUNT; i++) + for(ToolBox::ImplToolItems::size_type i = 0; i < NID_COUNT; i++) { sal_uInt16 nItemId = m_pToolBox->GetItemId( i ); m_pToolBox->CheckItem( nItemId, nItemId == nSet ); diff --git a/vcl/inc/toolbox.h b/vcl/inc/toolbox.h index 5ea8b50..f8f798d 100644 --- a/vcl/inc/toolbox.h +++ b/vcl/inc/toolbox.h @@ -112,7 +112,7 @@ struct ToolBoxLayoutData : public ControlLayoutData struct ImplToolBoxPrivateData { vcl::ToolBoxLayoutData* m_pLayoutData; - std::vector< ImplToolItem > m_aItems; + ToolBox::ImplToolItems m_aItems; ImplToolBoxPrivateData(); ~ImplToolBoxPrivateData(); diff --git a/vcl/osx/a11yfocustracker.cxx b/vcl/osx/a11yfocustracker.cxx index ad0f4f2..dc42f3e 100644 --- a/vcl/osx/a11yfocustracker.cxx +++ b/vcl/osx/a11yfocustracker.cxx @@ -119,9 +119,10 @@ void AquaA11yFocusTracker::notify_toolbox_item_focus(ToolBox *pToolBox) if( xContext.is() ) { - sal_Int32 nPos = pToolBox->GetItemPos( pToolBox->GetHighlightItemId() ); - if( nPos != TOOLBOX_ITEM_NOTFOUND ) + ToolBox::ImplToolItems::size_type nPos = pToolBox->GetItemPos( pToolBox->GetHighlightItemId() ); + if( nPos != ToolBox::ITEM_NOTFOUND ) setFocusedObject( xContext->getAccessibleChild( nPos ) ); + //TODO: ToolBox::ImplToolItems::size_type -> sal_Int32! } } } diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx index f5f0f7b..859d803 100644 --- a/vcl/source/window/builder.cxx +++ b/vcl/source/window/builder.cxx @@ -1707,6 +1707,7 @@ VclPtr<vcl::Window> VclBuilder::makeObject(vcl::Window *pParent, const OString & else { nItemId = pToolBox->GetItemCount() + 1; + //TODO: ImplToolItems::size_type -> sal_uInt16! pToolBox->InsertItem(nItemId, OStringToOUString(extractLabel(rMap), RTL_TEXTENCODING_UTF8), nBits); pToolBox->SetItemCommand(nItemId, aCommand); pToolBox->SetHelpId(nItemId, m_sHelpRoot + id); diff --git a/vcl/source/window/menubarwindow.cxx b/vcl/source/window/menubarwindow.cxx index dfca714..817e07e 100644 --- a/vcl/source/window/menubarwindow.cxx +++ b/vcl/source/window/menubarwindow.cxx @@ -67,8 +67,8 @@ void DecoToolBox::calcMinSize() } else { - sal_uInt16 nItems = GetItemCount(); - for( sal_uInt16 i = 0; i < nItems; i++ ) + ImplToolItems::size_type nItems = GetItemCount(); + for( ImplToolItems::size_type i = 0; i < nItems; i++ ) { sal_uInt16 nId = GetItemId( i ); aTbx->InsertItem( nId, GetItemImage( nId ) ); @@ -261,7 +261,7 @@ IMPL_LINK( MenuBarWindow, ToolboxEventHdl, VclWindowEvent&, rEvent, void ) aArg.nId = aCloseBtn->GetHighlightItemId(); else if( rEvent.GetId() == VclEventId::ToolboxHighlightOff ) { - sal_uInt16 nPos = static_cast< sal_uInt16 >(reinterpret_cast<sal_IntPtr>(rEvent.GetData())); + auto nPos = static_cast<ToolBox::ImplToolItems::size_type>(reinterpret_cast<sal_IntPtr>(rEvent.GetData())); aArg.nId = aCloseBtn->GetItemId(nPos); } std::map< sal_uInt16, AddButtonEntry >::iterator it = m_aAddButtons.find( aArg.nId ); @@ -1170,7 +1170,7 @@ Rectangle MenuBarWindow::GetMenuBarButtonRectPixel( sal_uInt16 nId ) void MenuBarWindow::RemoveMenuBarButton( sal_uInt16 nId ) { - sal_uInt16 nPos = aCloseBtn->GetItemPos(nId); + ToolBox::ImplToolItems::size_type nPos = aCloseBtn->GetItemPos(nId); aCloseBtn->RemoveItem(nPos); m_aAddButtons.erase( nId ); aCloseBtn->calcMinSize(); diff --git a/vcl/source/window/toolbox.cxx b/vcl/source/window/toolbox.cxx index f398969..033d6bd 100644 --- a/vcl/source/window/toolbox.cxx +++ b/vcl/source/window/toolbox.cxx @@ -654,7 +654,7 @@ static bool ImplIsFixedControl( const ImplToolItem *pItem ) const ImplToolItem *ToolBox::ImplGetFirstClippedItem( const ToolBox* pThis ) { - std::vector< ImplToolItem >::const_iterator it; + ImplToolItems::const_iterator it; it = pThis->mpData->m_aItems.begin(); while ( it != pThis->mpData->m_aItems.end() ) { @@ -789,7 +789,7 @@ void ToolBox::ImplCalcFloatSizes( ToolBox* pThis ) // calculate the minimal size, i.e. where the biggest item just fits long nCalcSize = 0; - std::vector< ImplToolItem >::const_iterator it; + ImplToolItems::const_iterator it; it = pThis->mpData->m_aItems.begin(); while ( it != pThis->mpData->m_aItems.end() ) { @@ -1078,7 +1078,7 @@ sal_uInt16 ToolBox::ImplFindItemPos( ToolBox* pBox, const Point& rPos ) aPos.Y() = aSize.Height()-TB_BORDER_OFFSET1; // Item suchen, das geklickt wurde - std::vector< ImplToolItem >::const_iterator it = pBox->mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = pBox->mpData->m_aItems.begin(); while ( it != pBox->mpData->m_aItems.end() ) { if ( it->mbVisible ) @@ -1360,7 +1360,7 @@ void ToolBox::ImplInitToolBoxData() mnHighItemId = 0; mnCurItemId = 0; mnDownItemId = 0; - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; mnLines = 1; mnCurLine = 1; mnCurLines = 1; @@ -1639,7 +1639,7 @@ ImplToolItem* ToolBox::ImplGetItem( sal_uInt16 nItemId ) const if (!mpData) return nullptr; - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { if ( it->mnId == nItemId ) @@ -1756,7 +1756,7 @@ bool ToolBox::ImplCalcItem() if ( ! mpData->m_aItems.empty() ) { - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { it->mbVisibleText = false; // indicates if text will definitely be drawn, influences dropdown pos @@ -1992,7 +1992,7 @@ sal_uInt16 ToolBox::ImplCalcBreaks( long nWidth, long* pMaxLineWidth, bool bCalc // we need to know which item is the last visible one to be able to add // the menu width in case we are unable to show all the items - std::vector< ImplToolItem >::iterator it, lastVisible; + ImplToolItems::iterator it, lastVisible; for ( it = mpData->m_aItems.begin(); it != mpData->m_aItems.end(); ++it ) { if ( it->mbVisible ) @@ -2184,10 +2184,10 @@ Size ToolBox::ImplGetOptimalFloatingSize() namespace { -void lcl_hideDoubleSeparators( std::vector< ImplToolItem >& rItems ) +void lcl_hideDoubleSeparators( ToolBox::ImplToolItems& rItems ) { bool bLastSep( true ); - std::vector< ImplToolItem >::iterator it; + ToolBox::ImplToolItems::iterator it; for ( it = rItems.begin(); it != rItems.end(); ++it ) { if ( it->meType == ToolBoxItemType::SEPARATOR ) @@ -2196,7 +2196,7 @@ void lcl_hideDoubleSeparators( std::vector< ImplToolItem >& rItems ) if ( !bLastSep ) { // check if any visible items have to appear behind it - std::vector< ImplToolItem >::iterator temp_it; + ToolBox::ImplToolItems::iterator temp_it; for ( temp_it = it+1; temp_it != rItems.end(); ++temp_it ) { if ( ((temp_it->meType == ToolBoxItemType::BUTTON) && @@ -2232,7 +2232,7 @@ void ToolBox::ImplFormat( bool bResize ) sal_uInt16 nFormatLine; bool bMustFullPaint; - std::vector< ImplToolItem >::iterator it; + ImplToolItems::iterator it; ImplDockingWindowWrapper *pWrapper = ImplGetDockingManager()->GetDockingWindowWrapper( this ); bool bIsInPopupMode = ImplIsInPopupMode(); @@ -2678,7 +2678,7 @@ void ToolBox::ImplFormat( bool bResize ) IMPL_LINK_NOARG(ToolBox, ImplDropdownLongClickHdl, Timer *, void) { - if (mnCurPos != TOOLBOX_ITEM_NOTFOUND && + if (mnCurPos != ITEM_NOTFOUND && (mpData->m_aItems[ mnCurPos ].mnBits & ToolBoxItemBits::DROPDOWN)) { mpData->mbDropDownByKeyboard = false; @@ -2692,7 +2692,7 @@ IMPL_LINK_NOARG(ToolBox, ImplDropdownLongClickHdl, Timer *, void) Deactivate(); InvalidateItem(mnCurPos); - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; mnCurItemId = 0; mnDownItemId = 0; mnMouseClicks = 0; @@ -2897,7 +2897,7 @@ void ToolBox::ImplDrawSpin(vcl::RenderContext& rRenderContext) false/*bUpperIn*/, false/*bLowerIn*/, bTmpUpper, bTmpLower, !mbHorz); } -void ToolBox::ImplDrawSeparator(vcl::RenderContext& rRenderContext, sal_uInt16 nPos, const Rectangle& rRect) +void ToolBox::ImplDrawSeparator(vcl::RenderContext& rRenderContext, ImplToolItems::size_type nPos, const Rectangle& rRect) { if ( nPos >= mpData->m_aItems.size() - 1 ) // no separator if it's the last item @@ -2970,7 +2970,7 @@ void ToolBox::ImplDrawButton(vcl::RenderContext& rRenderContext, const Rectangle bChecked, true, bIsWindow, nullptr, 2); } -void ToolBox::ImplDrawItem(vcl::RenderContext& rRenderContext, sal_uInt16 nPos, sal_uInt16 nHighlight) +void ToolBox::ImplDrawItem(vcl::RenderContext& rRenderContext, ImplToolItems::size_type nPos, sal_uInt16 nHighlight) { if (nPos >= mpData->m_aItems.size()) return; @@ -3347,13 +3347,13 @@ void ToolBox::ImplFloatControl( bool bStart, FloatingWindow* pFloatWindow ) // draw current item with highlight and keep old state bool bWasKeyboardActivate = mpData->mbDropDownByKeyboard; - if ( mnCurPos != TOOLBOX_ITEM_NOTFOUND ) + if ( mnCurPos != ITEM_NOTFOUND ) InvalidateItem(mnCurPos); Deactivate(); if( !bWasKeyboardActivate ) { - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; mnCurItemId = 0; mnHighItemId = 0; } @@ -3403,7 +3403,7 @@ bool ToolBox::ImplHandleMouseMove( const MouseEvent& rMEvt, bool bRepeat ) return false; // ToolBox active? - if ( mbDrag && mnCurPos != TOOLBOX_ITEM_NOTFOUND ) + if ( mbDrag && mnCurPos != ITEM_NOTFOUND ) { // is the cursor over the item? ImplToolItem* pItem = &mpData->m_aItems[mnCurPos]; @@ -3489,7 +3489,7 @@ bool ToolBox::ImplHandleMouseButtonUp( const MouseEvent& rMEvt, bool bCancel ) else { mbSelection = false; - if ( mnCurPos == TOOLBOX_ITEM_NOTFOUND ) + if ( mnCurPos == ITEM_NOTFOUND ) return true; } @@ -3540,7 +3540,7 @@ bool ToolBox::ImplHandleMouseButtonUp( const MouseEvent& rMEvt, bool bCancel ) // Get current pos for the case that items are inserted/removed // in the toolBox mnCurPos = GetItemPos( mnCurItemId ); - if ( mnCurPos != TOOLBOX_ITEM_NOTFOUND ) + if ( mnCurPos != ITEM_NOTFOUND ) { InvalidateItem(mnCurPos); Flush(); @@ -3549,7 +3549,7 @@ bool ToolBox::ImplHandleMouseButtonUp( const MouseEvent& rMEvt, bool bCancel ) } } - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; mnCurItemId = 0; mnDownItemId = 0; mnMouseClicks = 0; @@ -3594,11 +3594,11 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) if ( mbSelection && bDrawHotSpot ) { - sal_uInt16 i = 0; - sal_uInt16 nNewPos = TOOLBOX_ITEM_NOTFOUND; + ImplToolItems::size_type i = 0; + ImplToolItems::size_type nNewPos = ITEM_NOTFOUND; // search the item that has been clicked - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { // if the mouse position is in this item, @@ -3626,16 +3626,16 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) // was a new entry selected? // don't change selection if keyboard selection is active and // mouse leaves the toolbox - if ( nNewPos != mnCurPos && !( HasFocus() && nNewPos == TOOLBOX_ITEM_NOTFOUND ) ) + if ( nNewPos != mnCurPos && !( HasFocus() && nNewPos == ITEM_NOTFOUND ) ) { - if ( mnCurPos != TOOLBOX_ITEM_NOTFOUND ) + if ( mnCurPos != ITEM_NOTFOUND ) { InvalidateItem(mnCurPos); CallEventListeners( VclEventId::ToolboxHighlightOff, reinterpret_cast< void* >( mnCurPos ) ); } mnCurPos = nNewPos; - if ( mnCurPos != TOOLBOX_ITEM_NOTFOUND ) + if ( mnCurPos != ITEM_NOTFOUND ) { mnCurItemId = mnHighItemId = it->mnId; InvalidateItem(mnCurPos); @@ -3687,7 +3687,7 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) if ( (eStyle == PointerStyle::Arrow) && mbCustomizeMode ) { // search the item which was clicked - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { // show resize pointer if it is a customize window @@ -3708,9 +3708,9 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) if ( bDrawHotSpot && ( (mnOutStyle & TOOLBOX_STYLE_FLAT) || !mnOutStyle ) ) { bool bClearHigh = true; - if ( !rMEvt.IsLeaveWindow() && (mnCurPos == TOOLBOX_ITEM_NOTFOUND) ) + if ( !rMEvt.IsLeaveWindow() && (mnCurPos == ITEM_NOTFOUND) ) { - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { if ( it->maRect.IsInside( aMousePos ) ) @@ -3722,11 +3722,11 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) bClearHigh = false; if ( mnHighItemId != it->mnId ) { - sal_uInt16 nTempPos = sal::static_int_cast<sal_uInt16>(it - mpData->m_aItems.begin()); + ImplToolItems::size_type nTempPos = it - mpData->m_aItems.begin(); if ( mnHighItemId ) { ImplHideFocus(); - sal_uInt16 nPos = GetItemPos( mnHighItemId ); + ImplToolItems::size_type nPos = GetItemPos( mnHighItemId ); InvalidateItem(nPos); CallEventListeners( VclEventId::ToolboxHighlightOff, reinterpret_cast< void* >( nPos ) ); } @@ -3761,8 +3761,8 @@ void ToolBox::MouseMove( const MouseEvent& rMEvt ) if( mnHighItemId ) { - sal_uInt16 nClearPos = GetItemPos( mnHighItemId ); - if ( nClearPos != TOOLBOX_ITEM_NOTFOUND ) + ImplToolItems::size_type nClearPos = GetItemPos( mnHighItemId ); + if ( nClearPos != ITEM_NOTFOUND ) { InvalidateItem(nClearPos); if( nClearPos != mnCurPos ) @@ -3793,7 +3793,7 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt ) { // only trigger toolbox for left mouse button and when // we're not in normal operation - if ( rMEvt.IsLeft() && !mbDrag && (mnCurPos == TOOLBOX_ITEM_NOTFOUND) ) + if ( rMEvt.IsLeft() && !mbDrag && (mnCurPos == ITEM_NOTFOUND) ) { // call activate already here, as items could // be exchanged @@ -3807,11 +3807,11 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt ) } Point aMousePos = rMEvt.GetPosPixel(); - sal_uInt16 i = 0; - sal_uInt16 nNewPos = TOOLBOX_ITEM_NOTFOUND; + ImplToolItems::size_type i = 0; + ImplToolItems::size_type nNewPos = ITEM_NOTFOUND; // search for item that was clicked - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { // is this the item? @@ -3831,7 +3831,7 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt ) } // item found - if ( nNewPos != TOOLBOX_ITEM_NOTFOUND ) + if ( nNewPos != ITEM_NOTFOUND ) { if ( mbCustomize ) { @@ -3912,7 +3912,7 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt ) Deactivate(); InvalidateItem(mnCurPos); - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; mnCurItemId = 0; mnDownItemId = 0; mnMouseClicks = 0; @@ -4009,7 +4009,7 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt ) Click(); } - if ( !mbDrag && !mbSelection && (mnCurPos == TOOLBOX_ITEM_NOTFOUND) ) + if ( !mbDrag && !mbSelection && (mnCurPos == ITEM_NOTFOUND) ) DockingWindow::MouseButtonDown( rMEvt ); } @@ -4044,7 +4044,7 @@ void ToolBox::Tracking( const TrackingEvent& rTEvt ) DockingWindow::Tracking( rTEvt ); } -void ToolBox::InvalidateItem(sal_uInt16 nPosition) +void ToolBox::InvalidateItem(ImplToolItems::size_type nPosition) { if (mpData && nPosition < mpData->m_aItems.size()) { @@ -4096,14 +4096,14 @@ void ToolBox::Paint(vcl::RenderContext& rRenderContext, const Rectangle& rPaintR } // draw buttons - sal_uInt16 nHighPos; + ImplToolItems::size_type nHighPos; if ( mnHighItemId ) nHighPos = GetItemPos( mnHighItemId ); else - nHighPos = TOOLBOX_ITEM_NOTFOUND; + nHighPos = ITEM_NOTFOUND; - sal_uInt16 nCount = (sal_uInt16)mpData->m_aItems.size(); - for( sal_uInt16 i = 0; i < nCount; i++ ) + ImplToolItems::size_type nCount = mpData->m_aItems.size(); + for( ImplToolItems::size_type i = 0; i < nCount; i++ ) { ImplToolItem* pItem = &mpData->m_aItems[i]; @@ -4379,7 +4379,7 @@ bool ToolBox::EventNotify( NotifyEvent& rNEvt ) { // a child window got the focus so update current item to // allow for proper lose focus handling in keyboard navigation - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while( it != mpData->m_aItems.end() ) { if ( it->mbVisible ) @@ -4401,7 +4401,7 @@ bool ToolBox::EventNotify( NotifyEvent& rNEvt ) // deselect ImplHideFocus(); mnHighItemId = 0; - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; } return DockingWindow::EventNotify( rNEvt ); @@ -4412,7 +4412,7 @@ void ToolBox::Command( const CommandEvent& rCEvt ) // depict StartDrag on MouseButton/Left/Alt if ( (rCEvt.GetCommand() == CommandEventId::StartDrag) && rCEvt.IsMouseEvent() && mbCustomize && !mbDragging && !mbDrag && !mbSelection && - (mnCurPos == TOOLBOX_ITEM_NOTFOUND) ) + (mnCurPos == ITEM_NOTFOUND) ) { // We only allow dragging of items. Therefore, we have to check // if an item was clicked, otherwise we could move the window, and @@ -4422,7 +4422,7 @@ void ToolBox::Command( const CommandEvent& rCEvt ) if ( mbCustomizeMode ) { Point aMousePos = rCEvt.GetMousePosPixel(); - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); while ( it != mpData->m_aItems.end() ) { // is this the item? @@ -4540,7 +4540,7 @@ void ToolBox::statusChanged( const css::frame::FeatureStateEvent& Event ) // update image orientation OUString aModuleName(vcl::CommandInfoProvider::GetModuleIdentifier(mpStatusListener->getFrame())); - for (std::vector<ImplToolItem>::const_iterator it = mpData->m_aItems.begin(); it != mpData->m_aItems.end(); ++it) + for (ImplToolItems::const_iterator it = mpData->m_aItems.begin(); it != mpData->m_aItems.end(); ++it) { if (vcl::CommandInfoProvider::IsMirrored(it->maCommandStr, aModuleName)) SetItemImageMirrorMode(it->mnId, mbImagesMirrored); @@ -4840,7 +4840,7 @@ sal_uInt16 ToolBox::ImplCountLineBreaks( const ToolBox *pThis ) { sal_uInt16 nLines = 0; - std::vector< ImplToolItem >::const_iterator it = const_cast<ToolBox*>(pThis)->mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = const_cast<ToolBox*>(pThis)->mpData->m_aItems.begin(); while ( it != const_cast<ToolBox*>(pThis)->mpData->m_aItems.end() ) { if( it->meType == ToolBoxItemType::BREAK ) @@ -4909,7 +4909,7 @@ Size ToolBox::CalcMinimumWindowSizePixel() const VclPtrInstance< ToolBox > pToolBox( GetParent(), GetStyle() ); // copy until first useful item - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while( it != mpData->m_aItems.end() ) { pToolBox->CopyItem( *this, it->mnId ); @@ -5291,7 +5291,7 @@ void ToolBox::KeyInput( const KeyEvent& rKEvt ) // returns the current toolbox line of the item sal_uInt16 ToolBox::ImplGetItemLine( ImplToolItem* pCurrentItem ) { - std::vector< ImplToolItem >::const_iterator it = mpData->m_aItems.begin(); + ImplToolItems::const_iterator it = mpData->m_aItems.begin(); sal_uInt16 nLine = 1; while( it != mpData->m_aItems.end() ) { @@ -5312,7 +5312,7 @@ ImplToolItem* ToolBox::ImplGetFirstValidItem( sal_uInt16 nLine ) nLine--; - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while( it != mpData->m_aItems.end() ) { // find correct line @@ -5336,22 +5336,22 @@ ImplToolItem* ToolBox::ImplGetFirstValidItem( sal_uInt16 nLine ) return (it == mpData->m_aItems.end()) ? nullptr : &(*it); } -std::vector<ImplToolItem>::size_type ToolBox::ImplFindItemPos( const ImplToolItem* pItem, const std::vector< ImplToolItem >& rList ) +ToolBox::ImplToolItems::size_type ToolBox::ImplFindItemPos( const ImplToolItem* pItem, const ImplToolItems& rList ) { if( pItem ) { - for( std::vector<ImplToolItem>::size_type nPos = 0; nPos < rList.size(); ++nPos ) + for( ImplToolItems::size_type nPos = 0; nPos < rList.size(); ++nPos ) if( &rList[ nPos ] == pItem ) return nPos; } - return std::numeric_limits<std::vector<ImplToolItem>::size_type>::max(); + return ITEM_NOTFOUND; } -void ToolBox::ChangeHighlight( sal_uInt16 nPos ) +void ToolBox::ChangeHighlight( ImplToolItems::size_type nPos ) { if ( nPos < GetItemCount() ) { ImplGrabFocus( GetFocusFlags::NONE ); - ImplChangeHighlight ( ImplGetItem ( GetItemId ( (sal_uInt16) nPos ) ) ); + ImplChangeHighlight ( ImplGetItem ( GetItemId ( nPos ) ) ); } } @@ -5368,7 +5368,7 @@ void ToolBox::ImplChangeHighlight( ImplToolItem* pItem, bool bNoGrabFocus ) if ( mnHighItemId ) { ImplHideFocus(); - sal_uInt16 nPos = GetItemPos( mnHighItemId ); + ImplToolItems::size_type nPos = GetItemPos( mnHighItemId ); pOldItem = ImplGetItem( mnHighItemId ); // #i89962# ImplDrawItem can cause Invalidate/Update // which will in turn ImplShowFocus again @@ -5386,8 +5386,8 @@ void ToolBox::ImplChangeHighlight( ImplToolItem* pItem, bool bNoGrabFocus ) if( pItem ) { - std::vector<ImplToolItem>::size_type aPos = ToolBox::ImplFindItemPos( pItem, mpData->m_aItems ); - if( aPos != std::numeric_limits<std::vector<ImplToolItem>::size_type>::max()) + ImplToolItems::size_type aPos = ToolBox::ImplFindItemPos( pItem, mpData->m_aItems ); + if( aPos != ITEM_NOTFOUND) { // check for line breaks sal_uInt16 nLine = ImplGetItemLine( pItem ); @@ -5410,11 +5410,9 @@ void ToolBox::ImplChangeHighlight( ImplToolItem* pItem, bool bNoGrabFocus ) mnHighItemId = pItem->mnId; InvalidateItem(aPos); - //TODO: std::vector<ImplToolItem>::size_type -> sal_uInt16! if( mbSelection ) mnCurPos = aPos; - //TODO: std::vector<ImplToolItem>::size_type -> sal_uInt16! ImplShowFocus(); if( pItem->mpWindow ) @@ -5427,7 +5425,7 @@ void ToolBox::ImplChangeHighlight( ImplToolItem* pItem, bool bNoGrabFocus ) { ImplHideFocus(); mnHighItemId = 0; - mnCurPos = TOOLBOX_ITEM_NOTFOUND; + mnCurPos = ITEM_NOTFOUND; } mbChangingHighlight = false; @@ -5454,7 +5452,7 @@ bool ToolBox::ImplChangeHighlightUpDn( bool bUp, bool bNoCycle ) if( bUp ) { // select last valid non-clipped item - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.end(); + ImplToolItems::iterator it = mpData->m_aItems.end(); ImplToolItem* pItem = nullptr; while( it != mpData->m_aItems.begin() ) { @@ -5471,7 +5469,7 @@ bool ToolBox::ImplChangeHighlightUpDn( bool bUp, bool bNoCycle ) else { // select first valid non-clipped item - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while( it != mpData->m_aItems.end() ) { if ( ImplIsValidItem( &(*it), true ) ) @@ -5490,7 +5488,7 @@ bool ToolBox::ImplChangeHighlightUpDn( bool bUp, bool bNoCycle ) if( bUp ) { // Select first valid item - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.begin(); + ImplToolItems::iterator it = mpData->m_aItems.begin(); while( it != mpData->m_aItems.end() ) { if ( ImplIsValidItem( &(*it), false ) ) @@ -5520,7 +5518,7 @@ bool ToolBox::ImplChangeHighlightUpDn( bool bUp, bool bNoCycle ) } else { - std::vector< ImplToolItem >::iterator it = mpData->m_aItems.end(); + ImplToolItems::iterator it = mpData->m_aItems.end(); ImplToolItem* pItem = nullptr; while( it != mpData->m_aItems.begin() ) { @@ -5539,10 +5537,10 @@ bool ToolBox::ImplChangeHighlightUpDn( bool bUp, bool bNoCycle ) if( pToolItem ) { - std::vector<ImplToolItem>::size_type pos = ToolBox::ImplFindItemPos( pToolItem, mpData->m_aItems ); - std::vector<ImplToolItem>::size_type nCount = mpData->m_aItems.size(); + ImplToolItems::size_type pos = ToolBox::ImplFindItemPos( pToolItem, mpData->m_aItems ); + ImplToolItems::size_type nCount = mpData->m_aItems.size(); - std::vector<ImplToolItem>::size_type i=0; + ImplToolItems::size_type i=0; do { if( bUp ) diff --git a/vcl/source/window/toolbox2.cxx b/vcl/source/window/toolbox2.cxx index a4da7c3..2bba245 100644 --- a/vcl/source/window/toolbox2.cxx +++ b/vcl/source/window/toolbox2.cxx @@ -302,12 +302,12 @@ void ToolBox::ImplInvalidate( bool bNewCalc, bool bFullPaint ) CallEventListeners( VclEventId::ToolboxFormatChanged ); } -void ToolBox::ImplUpdateItem( sal_uInt16 nIndex ) +void ToolBox::ImplUpdateItem( ImplToolItems::size_type nIndex ) { // do we need to redraw? if ( IsReallyVisible() && IsUpdateMode() ) { - if ( nIndex == 0xFFFF ) + if ( nIndex == ITEM_NOTFOUND ) { // #i52217# no immediate draw as this might lead to paint problems Invalidate( Rectangle( mnLeftBorder, mnTopBorder, mnDX-mnRightBorder-1, mnDY-mnBottomBorder-1 ) ); @@ -372,10 +372,10 @@ void ToolBox::Select() pWrapper->GetFloatingWindow()->EndPopupMode(); } -void ToolBox::InsertItem( sal_uInt16 nItemId, const Image& rImage, ToolBoxItemBits nBits, sal_uInt16 nPos ) +void ToolBox::InsertItem( sal_uInt16 nItemId, const Image& rImage, ToolBoxItemBits nBits, ImplToolItems::size_type nPos ) { SAL_WARN_IF( !nItemId, "vcl", "ToolBox::InsertItem(): ItemId == 0" ); - SAL_WARN_IF( GetItemPos( nItemId ) != TOOLBOX_ITEM_NOTFOUND, "vcl", + SAL_WARN_IF( GetItemPos( nItemId ) != ITEM_NOTFOUND, "vcl", "ToolBox::InsertItem(): ItemId already exists" ); // create item and add to list @@ -387,15 +387,15 @@ void ToolBox::InsertItem( sal_uInt16 nItemId, const Image& rImage, ToolBoxItemBi ImplInvalidate( true ); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >(nNewPos ) ); } void ToolBox::InsertItem( sal_uInt16 nItemId, const Image& rImage, const OUString& rText, ToolBoxItemBits nBits, - sal_uInt16 nPos ) + ImplToolItems::size_type nPos ) { SAL_WARN_IF( !nItemId, "vcl", "ToolBox::InsertItem(): ItemId == 0" ); - SAL_WARN_IF( GetItemPos( nItemId ) != TOOLBOX_ITEM_NOTFOUND, "vcl", + SAL_WARN_IF( GetItemPos( nItemId ) != ITEM_NOTFOUND, "vcl", "ToolBox::InsertItem(): ItemId already exists" ); // create item and add to list @@ -407,14 +407,14 @@ void ToolBox::InsertItem( sal_uInt16 nItemId, const Image& rImage, const OUStrin ImplInvalidate( true ); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } -void ToolBox::InsertItem( sal_uInt16 nItemId, const OUString& rText, ToolBoxItemBits nBits, sal_uInt16 nPos ) +void ToolBox::InsertItem( sal_uInt16 nItemId, const OUString& rText, ToolBoxItemBits nBits, ImplToolItems::size_type nPos ) { SAL_WARN_IF( !nItemId, "vcl", "ToolBox::InsertItem(): ItemId == 0" ); - SAL_WARN_IF( GetItemPos( nItemId ) != TOOLBOX_ITEM_NOTFOUND, "vcl", + SAL_WARN_IF( GetItemPos( nItemId ) != ITEM_NOTFOUND, "vcl", "ToolBox::InsertItem(): ItemId already exists" ); // create item and add to list @@ -425,12 +425,12 @@ void ToolBox::InsertItem( sal_uInt16 nItemId, const OUString& rText, ToolBoxItem ImplInvalidate( true ); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } void ToolBox::InsertItem(const OUString& rCommand, const css::uno::Reference<css::frame::XFrame>& rFrame, ToolBoxItemBits nBits, - const Size& rRequestedSize, sal_uInt16 nPos) + const Size& rRequestedSize, ImplToolItems::size_type nPos) { OUString aModuleName(vcl::CommandInfoProvider::GetModuleIdentifier(rFrame)); OUString aLabel(vcl::CommandInfoProvider::GetLabelForCommand(rCommand, aModuleName)); @@ -446,6 +446,7 @@ void ToolBox::InsertItem(const OUString& rCommand, const css::uno::Reference<css Image aImage(CommandInfoProvider::GetImageForCommand(rCommand, rFrame, eImageType)); sal_uInt16 nItemId = GetItemCount() + 1; + //TODO: ImplToolItems::size_type -> sal_uInt16! InsertItem(nItemId, aImage, aLabel, nBits, nPos); SetItemCommand(nItemId, rCommand); SetQuickHelpText(nItemId, aTooltip); @@ -457,10 +458,10 @@ void ToolBox::InsertItem(const OUString& rCommand, const css::uno::Reference<css } void ToolBox::InsertWindow( sal_uInt16 nItemId, vcl::Window* pWindow, - ToolBoxItemBits nBits, sal_uInt16 nPos ) + ToolBoxItemBits nBits, ImplToolItems::size_type nPos ) { SAL_WARN_IF( !nItemId, "vcl", "ToolBox::InsertWindow(): ItemId == 0" ); - SAL_WARN_IF( GetItemPos( nItemId ) != TOOLBOX_ITEM_NOTFOUND, "vcl", + SAL_WARN_IF( GetItemPos( nItemId ) != ITEM_NOTFOUND, "vcl", "ToolBox::InsertWindow(): ItemId already exists" ); // create item and add to list @@ -478,7 +479,7 @@ void ToolBox::InsertWindow( sal_uInt16 nItemId, vcl::Window* pWindow, ImplInvalidate( true ); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } @@ -494,11 +495,11 @@ void ToolBox::InsertSpace() ImplInvalidate(); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(mpData->m_aItems.size() - 1); + ImplToolItems::size_type nNewPos = mpData->m_aItems.size() - 1; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } -void ToolBox::InsertSeparator( sal_uInt16 nPos, sal_uInt16 nPixSize ) +void ToolBox::InsertSeparator( ImplToolItems::size_type nPos, sal_uInt16 nPixSize ) { // create item and add to list ImplToolItem aItem; @@ -512,11 +513,11 @@ void ToolBox::InsertSeparator( sal_uInt16 nPos, sal_uInt16 nPixSize ) ImplInvalidate(); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } -void ToolBox::InsertBreak( sal_uInt16 nPos ) +void ToolBox::InsertBreak( ImplToolItems::size_type nPos ) { // create item and add to list ImplToolItem aItem; @@ -528,11 +529,11 @@ void ToolBox::InsertBreak( sal_uInt16 nPos ) ImplInvalidate(); // Notify - sal_uInt16 nNewPos = sal::static_int_cast<sal_uInt16>(( nPos == TOOLBOX_APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos); + ImplToolItems::size_type nNewPos = ( nPos == APPEND ) ? ( mpData->m_aItems.size() - 1 ) : nPos; CallEventListeners( VclEventId::ToolboxItemAdded, reinterpret_cast< void* >( nNewPos ) ); } -void ToolBox::RemoveItem( sal_uInt16 nPos ) +void ToolBox::RemoveItem( ImplToolItems::size_type nPos ) { if( nPos < mpData->m_aItems.size() ) { @@ -566,13 +567,13 @@ void ToolBox::RemoveItem( sal_uInt16 nPos ) void ToolBox::CopyItem( const ToolBox& rToolBox, sal_uInt16 nItemId ) { ... etc. - the rest is truncated _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits