diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-11 13:50:47 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-12 08:24:45 +0200 |
commit | 1c3ffcdd4e702264728d222046c401b90cb1c8c9 (patch) | |
tree | da40599532933bcc4f6363d56423150b07b2bc44 /framework/source | |
parent | 48ca5005ecea5286564c682f6d75716f3c4d8451 (diff) |
loplugin:constantparam
Change-Id: Ia769b3176aea287cd57414d7ce8e88c6da43883e
Reviewed-on: https://gerrit.libreoffice.org/55622
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework/source')
-rw-r--r-- | framework/source/uielement/statusbarmerger.cxx | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/framework/source/uielement/statusbarmerger.cxx b/framework/source/uielement/statusbarmerger.cxx index 4810152c02df..41719b2e24a8 100644 --- a/framework/source/uielement/statusbarmerger.cxx +++ b/framework/source/uielement/statusbarmerger.cxx @@ -102,14 +102,13 @@ bool lcl_MergeItems( StatusBar* pStatusbar, sal_uInt16 nPos, sal_uInt16 nModIndex, sal_uInt16& rItemId, - const ::rtl::OUString& rModuleIdentifier, const AddonStatusbarItemContainer& rAddonItems ) { const sal_uInt16 nSize( rAddonItems.size() ); for ( sal_Int32 i = 0; i < nSize; i++ ) { const AddonStatusbarItem& rItem = rAddonItems[i]; - if ( !StatusbarMerger::IsCorrectContext( rItem.aContext, rModuleIdentifier ) ) + if ( !StatusbarMerger::IsCorrectContext( rItem.aContext, OUString() ) ) continue; sal_uInt16 nInsPos = nPos + nModIndex + i; @@ -129,7 +128,7 @@ bool lcl_ReplaceItem( StatusBar* pStatusbar, const AddonStatusbarItemContainer& rAddonToolbarItems ) { pStatusbar->RemoveItem( pStatusbar->GetItemId( nPos ) ); - return lcl_MergeItems( pStatusbar, nPos, 0, rItemId, OUString(), rAddonToolbarItems ); + return lcl_MergeItems( pStatusbar, nPos, 0, rItemId, rAddonToolbarItems ); } bool lcl_RemoveItems( StatusBar* pStatusbar, @@ -194,9 +193,9 @@ bool StatusbarMerger::ProcessMergeOperation( const AddonStatusbarItemContainer& rItems ) { if ( rMergeCommand == MERGECOMMAND_ADDAFTER ) - return lcl_MergeItems( pStatusbar, nPos, 1, rItemId, "", rItems ); + return lcl_MergeItems( pStatusbar, nPos, 1, rItemId, rItems ); else if ( rMergeCommand == MERGECOMMAND_ADDBEFORE ) - return lcl_MergeItems( pStatusbar, nPos, 0, rItemId, "", rItems ); + return lcl_MergeItems( pStatusbar, nPos, 0, rItemId, rItems ); else if ( rMergeCommand == MERGECOMMAND_REPLACE ) return lcl_ReplaceItem( pStatusbar, nPos, rItemId, rItems ); else if ( rMergeCommand == MERGECOMMAND_REMOVE ) @@ -223,9 +222,9 @@ bool StatusbarMerger::ProcessMergeFallback( ( rMergeCommand == MERGECOMMAND_ADDAFTER ) ) { if ( rMergeFallback == "AddFirst" ) - return lcl_MergeItems( pStatusbar, 0, 0, rItemId, "", rItems ); + return lcl_MergeItems( pStatusbar, 0, 0, rItemId, rItems ); else if ( rMergeFallback == "AddLast" ) - return lcl_MergeItems( pStatusbar, STATUSBAR_APPEND, 0, rItemId, "", rItems ); + return lcl_MergeItems( pStatusbar, STATUSBAR_APPEND, 0, rItemId, rItems ); } return false; |