diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-19 17:44:29 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-20 09:19:54 +0200 |
commit | 039d9b97b720d9dc735413b292382104b63014d5 (patch) | |
tree | e366b57f1bf03ded002414f52bccddf98fb94e2e /framework/source/uielement | |
parent | c0565f7011fa6d6b295e522c023127970ccce822 (diff) |
loplugin:constparams in framework
Change-Id: Iecd19a0cbb3cc78158c17bfa064cc23bafc44b18
Reviewed-on: https://gerrit.libreoffice.org/40206
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework/source/uielement')
19 files changed, 27 insertions, 27 deletions
diff --git a/framework/source/uielement/addonstoolbarmanager.cxx b/framework/source/uielement/addonstoolbarmanager.cxx index 63b8c4729b89..f497ea01826e 100644 --- a/framework/source/uielement/addonstoolbarmanager.cxx +++ b/framework/source/uielement/addonstoolbarmanager.cxx @@ -102,7 +102,7 @@ static bool IsCorrectContext( const OUString& rModuleIdentifier, const OUString& return false; } -static Image RetrieveImage( Reference< css::frame::XFrame >& rFrame, +static Image RetrieveImage( Reference< css::frame::XFrame > const & rFrame, const OUString& aImageId, const OUString& aURL, bool bBigImage diff --git a/framework/source/uielement/comboboxtoolbarcontroller.cxx b/framework/source/uielement/comboboxtoolbarcontroller.cxx index 8f37f6d4f690..aeb35a07d6ae 100644 --- a/framework/source/uielement/comboboxtoolbarcontroller.cxx +++ b/framework/source/uielement/comboboxtoolbarcontroller.cxx @@ -191,7 +191,7 @@ void ComboboxToolbarController::LoseFocus() notifyFocusLost(); } -bool ComboboxToolbarController::PreNotify( NotifyEvent& rNEvt ) +bool ComboboxToolbarController::PreNotify( NotifyEvent const & rNEvt ) { switch ( rNEvt.GetType() ) { diff --git a/framework/source/uielement/controlmenucontroller.cxx b/framework/source/uielement/controlmenucontroller.cxx index b9975db9ade3..5cd2984fad64 100644 --- a/framework/source/uielement/controlmenucontroller.cxx +++ b/framework/source/uielement/controlmenucontroller.cxx @@ -159,7 +159,7 @@ private: }; void updateImagesPopupMenu( PopupMenu* pPopupMenu ); - void fillPopupMenu( uno::Reference< awt::XPopupMenu >& rPopupMenu ); + void fillPopupMenu( uno::Reference< awt::XPopupMenu > const & rPopupMenu ); bool m_bShowMenuImages : 1; std::unique_ptr<VclBuilder> m_xBuilder; @@ -191,7 +191,7 @@ void ControlMenuController::updateImagesPopupMenu( PopupMenu* pPopupMenu ) } // private function -void ControlMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void ControlMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pVCLPopupMenu = nullptr; diff --git a/framework/source/uielement/edittoolbarcontroller.cxx b/framework/source/uielement/edittoolbarcontroller.cxx index 023df2c2573f..e1e610bb4b75 100644 --- a/framework/source/uielement/edittoolbarcontroller.cxx +++ b/framework/source/uielement/edittoolbarcontroller.cxx @@ -169,7 +169,7 @@ void EditToolbarController::LoseFocus() notifyFocusLost(); } -bool EditToolbarController::PreNotify( NotifyEvent& rNEvt ) +bool EditToolbarController::PreNotify( NotifyEvent const & rNEvt ) { if( rNEvt.GetType() == MouseNotifyEvent::KEYINPUT ) { diff --git a/framework/source/uielement/fontmenucontroller.cxx b/framework/source/uielement/fontmenucontroller.cxx index 056ea97277ea..c1f64b64e259 100644 --- a/framework/source/uielement/fontmenucontroller.cxx +++ b/framework/source/uielement/fontmenucontroller.cxx @@ -72,7 +72,7 @@ FontMenuController::~FontMenuController() } // private function -void FontMenuController::fillPopupMenu( const Sequence< OUString >& rFontNameSeq, Reference< css::awt::XPopupMenu >& rPopupMenu ) +void FontMenuController::fillPopupMenu( const Sequence< OUString >& rFontNameSeq, Reference< css::awt::XPopupMenu > const & rPopupMenu ) { const OUString* pFontNameArray = rFontNameSeq.getConstArray(); VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); diff --git a/framework/source/uielement/fontsizemenucontroller.cxx b/framework/source/uielement/fontsizemenucontroller.cxx index 6977e827b3c1..53889e0785e4 100644 --- a/framework/source/uielement/fontsizemenucontroller.cxx +++ b/framework/source/uielement/fontsizemenucontroller.cxx @@ -68,7 +68,7 @@ FontSizeMenuController::~FontSizeMenuController() } // private function -OUString FontSizeMenuController::retrievePrinterName( css::uno::Reference< css::frame::XFrame >& rFrame ) +OUString FontSizeMenuController::retrievePrinterName( css::uno::Reference< css::frame::XFrame > const & rFrame ) { OUString aPrinterName; @@ -97,7 +97,7 @@ OUString FontSizeMenuController::retrievePrinterName( css::uno::Reference< css:: } // private function -void FontSizeMenuController::setCurHeight( long nHeight, Reference< css::awt::XPopupMenu >& rPopupMenu ) +void FontSizeMenuController::setCurHeight( long nHeight, Reference< css::awt::XPopupMenu > const & rPopupMenu ) { // check menu item sal_uInt16 nChecked = 0; @@ -121,7 +121,7 @@ void FontSizeMenuController::setCurHeight( long nHeight, Reference< css::awt::XP } // private function -void FontSizeMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void FontSizeMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pVCLPopupMenu = nullptr; diff --git a/framework/source/uielement/headermenucontroller.cxx b/framework/source/uielement/headermenucontroller.cxx index aa6ad944faed..41bf068c7047 100644 --- a/framework/source/uielement/headermenucontroller.cxx +++ b/framework/source/uielement/headermenucontroller.cxx @@ -71,7 +71,7 @@ HeaderMenuController::~HeaderMenuController() } // private function -void HeaderMenuController::fillPopupMenu( const Reference< css::frame::XModel >& rModel, Reference< css::awt::XPopupMenu >& rPopupMenu ) +void HeaderMenuController::fillPopupMenu( const Reference< css::frame::XModel >& rModel, Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pVCLPopupMenu = nullptr; diff --git a/framework/source/uielement/langselectionmenucontroller.cxx b/framework/source/uielement/langselectionmenucontroller.cxx index e8c1c11a33c8..701bdf1902da 100644 --- a/framework/source/uielement/langselectionmenucontroller.cxx +++ b/framework/source/uielement/langselectionmenucontroller.cxx @@ -154,7 +154,7 @@ void LanguageSelectionMenuController::impl_setPopupMenu() m_xMenuDispatch_CharDlgForParagraph = xDispatchProvider->queryDispatch( aTargetURL, OUString(), 0 ); } -void LanguageSelectionMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu , const Mode eMode ) +void LanguageSelectionMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu , const Mode eMode ) { VCLXPopupMenu* pVCLPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pPopupMenu = nullptr; diff --git a/framework/source/uielement/macrosmenucontroller.cxx b/framework/source/uielement/macrosmenucontroller.cxx index 624b56395c00..7b54f2aaa911 100644 --- a/framework/source/uielement/macrosmenucontroller.cxx +++ b/framework/source/uielement/macrosmenucontroller.cxx @@ -65,7 +65,7 @@ MacrosMenuController::~MacrosMenuController() } // private function -void MacrosMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void MacrosMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pVCLPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pPopupMenu = nullptr; diff --git a/framework/source/uielement/menubarmanager.cxx b/framework/source/uielement/menubarmanager.cxx index ea28334b7768..0b82b567c855 100644 --- a/framework/source/uielement/menubarmanager.cxx +++ b/framework/source/uielement/menubarmanager.cxx @@ -1923,7 +1923,7 @@ void MenuBarManager::UpdateSpecialWindowMenu( Menu* pMenu,const Reference< XComp } } -void MenuBarManager::FillMenuImages(Reference< XFrame >& _xFrame, Menu* _pMenu,bool bShowMenuImages) +void MenuBarManager::FillMenuImages(Reference< XFrame > const & _xFrame, Menu* _pMenu,bool bShowMenuImages) { AddonsOptions aAddonOptions; diff --git a/framework/source/uielement/menubarmerger.cxx b/framework/source/uielement/menubarmerger.cxx index 7a9123cb96bb..eb478b70283a 100644 --- a/framework/source/uielement/menubarmerger.cxx +++ b/framework/source/uielement/menubarmerger.cxx @@ -142,7 +142,7 @@ ReferencePathInfo MenuBarMerger::FindReferencePath( return aResult; } -sal_uInt16 MenuBarMerger::FindMenuItem( const OUString& rCmd, Menu* pCurrMenu ) +sal_uInt16 MenuBarMerger::FindMenuItem( const OUString& rCmd, Menu const * pCurrMenu ) { for ( sal_uInt16 i = 0; i < pCurrMenu->GetItemCount(); i++ ) { diff --git a/framework/source/uielement/newmenucontroller.cxx b/framework/source/uielement/newmenucontroller.cxx index ddc51da96d3b..adb39d870bf6 100644 --- a/framework/source/uielement/newmenucontroller.cxx +++ b/framework/source/uielement/newmenucontroller.cxx @@ -300,7 +300,7 @@ NewMenuController::~NewMenuController() } // private function -void NewMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void NewMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pVCLPopupMenu = nullptr; diff --git a/framework/source/uielement/notebookbarmenucontroller.cxx b/framework/source/uielement/notebookbarmenucontroller.cxx index 70d7827dd7e2..292fd3b7beb4 100644 --- a/framework/source/uielement/notebookbarmenucontroller.cxx +++ b/framework/source/uielement/notebookbarmenucontroller.cxx @@ -75,7 +75,7 @@ NotebookbarMenuController::~NotebookbarMenuController() { } -void NotebookbarMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void NotebookbarMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { if ( SvtMiscOptions().DisableUICustomization() ) return; diff --git a/framework/source/uielement/objectmenucontroller.cxx b/framework/source/uielement/objectmenucontroller.cxx index 0cba9ca7f148..627ae94114c4 100644 --- a/framework/source/uielement/objectmenucontroller.cxx +++ b/framework/source/uielement/objectmenucontroller.cxx @@ -77,7 +77,7 @@ public: virtual void SAL_CALL disposing( const css::lang::EventObject& Source ) override; private: - void fillPopupMenu( const css::uno::Sequence< css::embed::VerbDescriptor >& rVerbCommandSeq, css::uno::Reference< css::awt::XPopupMenu >& rPopupMenu ); + void fillPopupMenu( const css::uno::Sequence< css::embed::VerbDescriptor >& rVerbCommandSeq, css::uno::Reference< css::awt::XPopupMenu > const & rPopupMenu ); }; ObjectMenuController::ObjectMenuController( const css::uno::Reference< css::uno::XComponentContext >& xContext ) : @@ -86,7 +86,7 @@ ObjectMenuController::ObjectMenuController( const css::uno::Reference< css::uno: } // private function -void ObjectMenuController::fillPopupMenu( const Sequence< css::embed::VerbDescriptor >& rVerbCommandSeq, Reference< css::awt::XPopupMenu >& rPopupMenu ) +void ObjectMenuController::fillPopupMenu( const Sequence< css::embed::VerbDescriptor >& rVerbCommandSeq, Reference< css::awt::XPopupMenu > const & rPopupMenu ) { const css::embed::VerbDescriptor* pVerbCommandArray = rVerbCommandSeq.getConstArray(); VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); diff --git a/framework/source/uielement/recentfilesmenucontroller.cxx b/framework/source/uielement/recentfilesmenucontroller.cxx index a4d967c2bb76..6d3f2deafb5c 100644 --- a/framework/source/uielement/recentfilesmenucontroller.cxx +++ b/framework/source/uielement/recentfilesmenucontroller.cxx @@ -89,7 +89,7 @@ public: private: virtual void impl_setPopupMenu() override; - void fillPopupMenu( css::uno::Reference< css::awt::XPopupMenu >& rPopupMenu ); + void fillPopupMenu( css::uno::Reference< css::awt::XPopupMenu > const & rPopupMenu ); void executeEntry( sal_Int32 nIndex ); std::vector< OUString > m_aRecentFilesItems; @@ -116,7 +116,7 @@ RecentFilesMenuController::RecentFilesMenuController( const uno::Reference< uno: } // private function -void RecentFilesMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void RecentFilesMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { VCLXPopupMenu* pPopupMenu = static_cast<VCLXPopupMenu *>(VCLXMenu::GetImplementation( rPopupMenu )); PopupMenu* pVCLPopupMenu = nullptr; diff --git a/framework/source/uielement/spinfieldtoolbarcontroller.cxx b/framework/source/uielement/spinfieldtoolbarcontroller.cxx index 3d45fb7be8b1..f0c2d11cbddb 100644 --- a/framework/source/uielement/spinfieldtoolbarcontroller.cxx +++ b/framework/source/uielement/spinfieldtoolbarcontroller.cxx @@ -261,7 +261,7 @@ void SpinfieldToolbarController::GetFocus() notifyFocusGet(); } -bool SpinfieldToolbarController::PreNotify( NotifyEvent& rNEvt ) +bool SpinfieldToolbarController::PreNotify( NotifyEvent const & rNEvt ) { if( rNEvt.GetType() == MouseNotifyEvent::KEYINPUT ) { diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx index 584aabdb7ec6..ae92672d40ec 100644 --- a/framework/source/uielement/toolbarmanager.cxx +++ b/framework/source/uielement/toolbarmanager.cxx @@ -93,7 +93,7 @@ static const char ITEM_DESCRIPTOR_VISIBLE[] = "IsVisible"; static const sal_uInt16 STARTID_CUSTOMIZE_POPUPMENU = 1000; static css::uno::Reference< css::frame::XLayoutManager > getLayoutManagerFromFrame( - css::uno::Reference< css::frame::XFrame >& rFrame ) + css::uno::Reference< css::frame::XFrame > const & rFrame ) { css::uno::Reference< css::frame::XLayoutManager > xLayoutManager; @@ -1177,7 +1177,7 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine } } -void ToolBarManager::FillOverflowToolbar( ToolBox* pParent ) +void ToolBarManager::FillOverflowToolbar( ToolBox const * pParent ) { CommandInfo aCmdInfo; bool bInsertSeparator = false; @@ -1401,7 +1401,7 @@ bool ToolBarManager::MenuItemAllowed( sal_uInt16 ) const return true; } -void ToolBarManager::AddCustomizeMenuItems(ToolBox* pToolBar) +void ToolBarManager::AddCustomizeMenuItems(ToolBox const * pToolBar) { // No config menu entries if command ".uno:ConfigureDialog" is not enabled Reference< XDispatch > xDisp; diff --git a/framework/source/uielement/toolbarmodemenucontroller.cxx b/framework/source/uielement/toolbarmodemenucontroller.cxx index 2ac7f4671df8..54d74439261b 100644 --- a/framework/source/uielement/toolbarmodemenucontroller.cxx +++ b/framework/source/uielement/toolbarmodemenucontroller.cxx @@ -75,7 +75,7 @@ ToolbarModeMenuController::~ToolbarModeMenuController() { } -void ToolbarModeMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void ToolbarModeMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { if ( SvtMiscOptions().DisableUICustomization() ) return; diff --git a/framework/source/uielement/toolbarsmenucontroller.cxx b/framework/source/uielement/toolbarsmenucontroller.cxx index c5775e88d7b6..1b5c451af138 100644 --- a/framework/source/uielement/toolbarsmenucontroller.cxx +++ b/framework/source/uielement/toolbarsmenucontroller.cxx @@ -136,7 +136,7 @@ ToolbarsMenuController::~ToolbarsMenuController() } void ToolbarsMenuController::addCommand( - Reference< css::awt::XPopupMenu >& rPopupMenu, const OUString& rCommandURL, const OUString& rLabel ) + Reference< css::awt::XPopupMenu > const & rPopupMenu, const OUString& rCommandURL, const OUString& rLabel ) { sal_uInt16 nItemId = m_xPopupMenu->getItemCount()+1; @@ -275,7 +275,7 @@ Sequence< Sequence< css::beans::PropertyValue > > ToolbarsMenuController::getLay } -void ToolbarsMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& rPopupMenu ) +void ToolbarsMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu > const & rPopupMenu ) { if( SvtMiscOptions().DisableUICustomization() ) return; |