loplugin:constparams improve handling of pointer params

Change-Id: I4c0002e72703eded435bfe4985f5b0121bf8524b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122843
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

5 files changed, 10 insertions, 10 deletions

diff --git a/cui/source/dialogs/scriptdlg.cxx b/cui/source/dialogs/scriptdlg.cxx
index 1258ee54d432..f4d347b0ef84 100644
--- a/cui/source/dialogs/scriptdlg.cxx
+++ b/cui/source/dialogs/scriptdlg.cxx
@@ -75,7 +75,7 @@ void SvxScriptOrgDialog::delUserData(const weld::TreeIter& rIter)
     }
 }
 
-void SvxScriptOrgDialog::deleteTree(weld::TreeIter& rIter)
+void SvxScriptOrgDialog::deleteTree(const weld::TreeIter& rIter)
 {
     delUserData(rIter);
     std::unique_ptr<weld::TreeIter> xIter = m_xScriptsBox->make_iterator(&rIter);
@@ -697,7 +697,7 @@ Reference< XModel > SvxScriptOrgDialog::getModel(const weld::TreeIter& rEntry)
     return model;
 }
 
-void SvxScriptOrgDialog::createEntry(weld::TreeIter& rEntry)
+void SvxScriptOrgDialog::createEntry(const weld::TreeIter& rEntry)
 {
 
     Reference< browse::XBrowseNode >  aChildNode;
@@ -927,7 +927,7 @@ void SvxScriptOrgDialog::renameEntry(const weld::TreeIter& rEntry)
     }
 }
 
-void SvxScriptOrgDialog::deleteEntry(weld::TreeIter& rEntry)
+void SvxScriptOrgDialog::deleteEntry(const weld::TreeIter& rEntry)
 {
     bool result = false;
     Reference< browse::XBrowseNode > node = getBrowseNode(rEntry);
diff --git a/cui/source/inc/numfmt.hxx b/cui/source/inc/numfmt.hxx
index b4bddde39aaa..13f1a88e9eab 100644
--- a/cui/source/inc/numfmt.hxx
+++ b/cui/source/inc/numfmt.hxx
@@ -129,7 +129,7 @@ private:
     void    MakePreviewText( const OUString& rFormat );
     void    ChangePreviewText( sal_uInt16 nPos );
     void    AddAutomaticLanguage_Impl(LanguageType eAutoLang, bool bSelect);
-    bool    Click_Impl(weld::Button& rIB);
+    bool    Click_Impl(const weld::Button& rIB);
     // Handler
     DECL_LINK(LostFocusHdl_Impl, weld::Widget&, void);
     DECL_LINK(DoubleClickHdl_Impl, weld::TreeView&, bool);
diff --git a/cui/source/inc/scriptdlg.hxx b/cui/source/inc/scriptdlg.hxx
index 6a1cbd15ae5c..3f16d8a66bab 100644
--- a/cui/source/inc/scriptdlg.hxx
+++ b/cui/source/inc/scriptdlg.hxx
@@ -106,9 +106,9 @@ protected:
     static bool         getBoolProperty( css::uno::Reference< css::beans::XPropertySet > const & xProps, OUString const & propName );
     void                CheckButtons(  css::uno::Reference< css::script::browse::XBrowseNode > const & node );
 
-    void        createEntry(weld::TreeIter& rEntry);
+    void        createEntry(const weld::TreeIter& rEntry);
     void        renameEntry(const weld::TreeIter& rEntry);
-    void        deleteEntry(weld::TreeIter& rEntry);
+    void        deleteEntry(const weld::TreeIter& rEntry);
     css::uno::Reference<css::script::browse::XBrowseNode> getBrowseNode(const weld::TreeIter& rEntry);
     css::uno::Reference<css::frame::XModel> getModel(const weld::TreeIter& rEntry);
     OUString    getListOfChildren( const css::uno::Reference< css::script::browse::XBrowseNode >& node, int depth );
@@ -117,7 +117,7 @@ protected:
 
     void        Init(std::u16string_view language);
     void        delUserData(const weld::TreeIter& rIter);
-    void        deleteTree(weld::TreeIter& rIter);
+    void        deleteTree(const weld::TreeIter& rIter);
     void        deleteAllTree();
     void        insertEntry(OUString const & rText, OUString const & rBitmap,
                               const weld::TreeIter* pParent,
diff --git a/cui/source/options/optcolor.cxx b/cui/source/options/optcolor.cxx
index 9ab34dd61434..ebe066b1073e 100644
--- a/cui/source/options/optcolor.cxx
+++ b/cui/source/options/optcolor.cxx
@@ -170,7 +170,7 @@ public:
                   Link<weld::Widget&,void> const&,
                   weld::ScrolledWindow& rScroll);
     void Update(EditableColorConfig const*, EditableExtendedColorConfig const*);
-    void ClickHdl(EditableColorConfig*, weld::Toggleable&);
+    void ClickHdl(EditableColorConfig*, const weld::Toggleable&);
     void ColorHdl(EditableColorConfig*, EditableExtendedColorConfig*, const ColorListBox*);
 
     weld::Widget& GetWidget1()
@@ -496,7 +496,7 @@ void ColorConfigWindow_Impl::Update (
 }
 
 // ClickHdl()
-void ColorConfigWindow_Impl::ClickHdl(EditableColorConfig* pConfig, weld::Toggleable& rBox)
+void ColorConfigWindow_Impl::ClickHdl(EditableColorConfig* pConfig, const weld::Toggleable& rBox)
 {
     for (unsigned i = 0; i != ColorConfigEntryCount; ++i)
     {
diff --git a/cui/source/tabpages/numfmt.cxx b/cui/source/tabpages/numfmt.cxx
index a04aaf767afd..21f60f29e598 100644
--- a/cui/source/tabpages/numfmt.cxx
+++ b/cui/source/tabpages/numfmt.cxx
@@ -1288,7 +1288,7 @@ IMPL_LINK( SvxNumberFormatTabPage, ClickHdl_Impl, weld::Button&, rIB, void)
     Click_Impl(rIB);
 }
 
-bool SvxNumberFormatTabPage::Click_Impl(weld::Button& rIB)
+bool SvxNumberFormatTabPage::Click_Impl(const weld::Button& rIB)
 {
     sal_uLong       nReturn = 0;
     constexpr sal_uLong nReturnChanged  = 0x1;  // THE boolean return value