summaryrefslogtreecommitdiff
path: root/framework
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2020-04-21 12:11:10 +0100
committerCaolán McNamara <caolanm@redhat.com>2020-04-22 09:38:48 +0200
commitf8156f7e81145cbf1c287058039ae70585cf64e7 (patch)
tree3ca2bd14811f68bba348d42d9d87369283d90b26 /framework
parentffb4ce52ea32fbaf4944a5b5072d6385eccf3cb3 (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.cxx5
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++ )
{