diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-04-21 12:11:10 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2020-04-22 09:38:48 +0200 |
commit | f8156f7e81145cbf1c287058039ae70585cf64e7 (patch) | |
tree | 3ca2bd14811f68bba348d42d9d87369283d90b26 /framework | |
parent | ffb4ce52ea32fbaf4944a5b5072d6385eccf3cb3 (diff) |
seems a poor idea to increment a vector iter by COMBOBOX_APPEND
which is SAL_MAX_INT32, in the (presumably never happens) fallback case
Change-Id: If425c6ce1bd9937ba2b26f6c68231640a2f13273
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92626
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'framework')
-rw-r--r-- | framework/source/uielement/togglebuttontoolbarcontroller.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/framework/source/uielement/togglebuttontoolbarcontroller.cxx b/framework/source/uielement/togglebuttontoolbarcontroller.cxx index ca2d4312c443..0f0e341d1448 100644 --- a/framework/source/uielement/togglebuttontoolbarcontroller.cxx +++ b/framework/source/uielement/togglebuttontoolbarcontroller.cxx @@ -22,7 +22,6 @@ #include <vcl/svapp.hxx> #include <vcl/toolbox.hxx> #include <vcl/menu.hxx> -#include <vcl/combobox.hxx> using namespace ::com::sun::star; using namespace ::com::sun::star::awt; @@ -184,8 +183,8 @@ void ToggleButtonToolbarController::executeControlCommand( const css::frame::Con } else if ( rControlCommand.Command == "InsertEntry" ) { - sal_Int32 nPos( COMBOBOX_APPEND ); - sal_Int32 nSize = sal_Int32( m_aDropdownMenuList.size() ); + sal_Int32 nPos(0); + sal_Int32 nSize = sal_Int32( m_aDropdownMenuList.size() ); OUString aText; for ( sal_Int32 i = 0; i < rControlCommand.Arguments.getLength(); i++ ) { |