chart2/source/model/main/Diagram.cxx | 4 ++-- chart2/source/view/charttypes/PieChart.cxx | 2 +- chart2/source/view/inc/DataTableView.hxx | 8 ++++---- chart2/source/view/main/DataTableView.cxx | 10 +++++----- include/svl/itempool.hxx | 2 +- include/xmloff/txtimp.hxx | 2 +- svl/source/items/itempool.cxx | 2 +- xmloff/inc/XMLThemeContext.hxx | 2 +- xmloff/source/style/XMLThemeContext.cxx | 2 +- xmloff/source/text/txtimp.cxx | 2 +- 10 files changed, 18 insertions(+), 18 deletions(-)
New commits: commit a94c1371cfa2051b9bbb30166217ccedfa274f58 Author: Noel Grandin <[email protected]> AuthorDate: Tue Mar 25 09:53:12 2025 +0200 Commit: Noel Grandin <[email protected]> CommitDate: Tue Mar 25 11:44:48 2025 +0100 loplugin:constparam in xmloff Change-Id: I24c536279c5b02b32d564893793fa15672c6f73c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183288 Tested-by: Jenkins Reviewed-by: Noel Grandin <[email protected]> diff --git a/include/xmloff/txtimp.hxx b/include/xmloff/txtimp.hxx index 061b2e3b34ba..ea6515a952d9 100644 --- a/include/xmloff/txtimp.hxx +++ b/include/xmloff/txtimp.hxx @@ -168,7 +168,7 @@ public: // Add parameter <bOutlineLevelAttrFound> (#i73509#) // Add parameter <bSetListAttrs> in order to suppress the handling of the list attributes (#i80724#) OUString SetStyleAndAttrs( - SvXMLImport & rImport, + const SvXMLImport & rImport, const css::uno::Reference< css::text::XTextCursor >& rCursor, const OUString& rStyleName, bool bPara, diff --git a/xmloff/inc/XMLThemeContext.hxx b/xmloff/inc/XMLThemeContext.hxx index 7b24671eb3ff..c699673064a1 100644 --- a/xmloff/inc/XMLThemeContext.hxx +++ b/xmloff/inc/XMLThemeContext.hxx @@ -61,7 +61,7 @@ class XMLColorContext : public SvXMLImportContext public: XMLColorContext(SvXMLImport& rImport, css::uno::Reference<css::xml::sax::XFastAttributeList> const& xAttrList, - std::shared_ptr<model::ColorSet>& rpColorSet); + const std::shared_ptr<model::ColorSet>& rpColorSet); }; /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/xmloff/source/style/XMLThemeContext.cxx b/xmloff/source/style/XMLThemeContext.cxx index 971c2eaddeb0..5d0f516faafb 100644 --- a/xmloff/source/style/XMLThemeContext.cxx +++ b/xmloff/source/style/XMLThemeContext.cxx @@ -107,7 +107,7 @@ uno::Reference<xml::sax::XFastContextHandler> XMLColorContext::XMLColorContext(SvXMLImport& rImport, const uno::Reference<xml::sax::XFastAttributeList>& xAttrList, - std::shared_ptr<model::ColorSet>& rpColorSet) + const std::shared_ptr<model::ColorSet>& rpColorSet) : SvXMLImportContext(rImport) { OUString aName; diff --git a/xmloff/source/text/txtimp.cxx b/xmloff/source/text/txtimp.cxx index 33fa6ab0ae0f..43dade94fcca 100644 --- a/xmloff/source/text/txtimp.cxx +++ b/xmloff/source/text/txtimp.cxx @@ -1003,7 +1003,7 @@ static bool lcl_HasListStyle( const OUString& sStyleName, } OUString XMLTextImportHelper::SetStyleAndAttrs( - SvXMLImport & rImport, + const SvXMLImport & rImport, const Reference < XTextCursor >& rCursor, const OUString& rStyleName, bool bPara, commit 8944b5872403d58f0780059b35f16aa5cc4951de Author: Noel Grandin <[email protected]> AuthorDate: Tue Mar 25 09:52:53 2025 +0200 Commit: Noel Grandin <[email protected]> CommitDate: Tue Mar 25 11:44:41 2025 +0100 loplugin:constparam in svl Change-Id: Ib975b84d73d6a7012a3d20c69872a0b0039f7929 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183287 Reviewed-by: Noel Grandin <[email protected]> Tested-by: Jenkins diff --git a/include/svl/itempool.hxx b/include/svl/itempool.hxx index de43d4c5355e..5498197eafa0 100644 --- a/include/svl/itempool.hxx +++ b/include/svl/itempool.hxx @@ -101,7 +101,7 @@ class UNLESS_MERGELIBS(SVL_DLLPUBLIC) ItemInfoUser : public ItemInfo const SfxPoolItem* m_pItem; public: - ItemInfoUser(const ItemInfo& rItemInfo, SfxItemPool& rItemPool, const SfxPoolItem& rItem, bool bPassingOwnership = false); + ItemInfoUser(const ItemInfo& rItemInfo, const SfxItemPool& rItemPool, const SfxPoolItem& rItem, bool bPassingOwnership = false); virtual ~ItemInfoUser(); virtual const SfxPoolItem* getItem() const override { return m_pItem; } diff --git a/svl/source/items/itempool.cxx b/svl/source/items/itempool.cxx index 651ba1277433..db62b26e8955 100644 --- a/svl/source/items/itempool.cxx +++ b/svl/source/items/itempool.cxx @@ -186,7 +186,7 @@ // RES_UNKNOWNATR_CONTAINER ok // RES_TXTATR_UNKNOWN_CONTAINER ok -ItemInfoUser::ItemInfoUser(const ItemInfo& rItemInfo, SfxItemPool& rItemPool, const SfxPoolItem& rItem, bool bPassingOwnership) +ItemInfoUser::ItemInfoUser(const ItemInfo& rItemInfo, const SfxItemPool& rItemPool, const SfxPoolItem& rItem, bool bPassingOwnership) : ItemInfo(rItemInfo) , m_pItem(implCreateItemEntry(rItemPool, &rItem, bPassingOwnership)) { commit c23d7d4dda56bc35a7ddf2fb0219d4c026187d18 Author: Noel Grandin <[email protected]> AuthorDate: Tue Mar 25 09:52:39 2025 +0200 Commit: Noel Grandin <[email protected]> CommitDate: Tue Mar 25 11:44:34 2025 +0100 loplugin:constparam in chart2 Change-Id: I3d538fe3e97569e9c67c7924ffa585c93d131dc1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183286 Tested-by: Jenkins Reviewed-by: Noel Grandin <[email protected]> diff --git a/chart2/source/model/main/Diagram.cxx b/chart2/source/model/main/Diagram.cxx index c3c4e0be9cc6..4466b752e9b8 100644 --- a/chart2/source/model/main/Diagram.cxx +++ b/chart2/source/model/main/Diagram.cxx @@ -1015,7 +1015,7 @@ bool Diagram::isSupportingFloorAndWall() * */ static bool lcl_moveSeriesOrCheckIfMoveIsAllowed( - Diagram& rDiagram, + const Diagram& rDiagram, const rtl::Reference< DataSeries >& xGivenDataSeries, bool bForward, bool bDoMove ) @@ -1210,7 +1210,7 @@ bool Diagram::isSupportingDateAxis() } static std::vector< rtl::Reference< Axis > > lcl_getAxisHoldingCategoriesFromDiagram( - Diagram& rDiagram ) + const Diagram& rDiagram ) { std::vector< rtl::Reference< Axis > > aRet; diff --git a/chart2/source/view/charttypes/PieChart.cxx b/chart2/source/view/charttypes/PieChart.cxx index 82ed7f32f6d4..e801cb120dbf 100644 --- a/chart2/source/view/charttypes/PieChart.cxx +++ b/chart2/source/view/charttypes/PieChart.cxx @@ -1181,7 +1181,7 @@ static sal_Int32 propIndex( sal_Int32 nPointIndex, enum SubPieType eType, const PieDataSrcBase *pDataSrc, - VDataSeries* pSeries) + const VDataSeries* pSeries) { switch (eType) { diff --git a/chart2/source/view/inc/DataTableView.hxx b/chart2/source/view/inc/DataTableView.hxx index 0bccaaddde31..f6a1ab2c97b6 100644 --- a/chart2/source/view/inc/DataTableView.hxx +++ b/chart2/source/view/inc/DataTableView.hxx @@ -54,14 +54,14 @@ private: bool m_bAlignAxisValuesWithColumns; /** Set the char and paragraph properties for the input (value) cell */ - void - setCellCharAndParagraphProperties(css::uno::Reference<css::beans::XPropertySet>& xPropertySet); + void setCellCharAndParagraphProperties( + const css::uno::Reference<css::beans::XPropertySet>& xPropertySet); /** Set the common cell properties (for all cells in the data table, * including headers) */ - void setCellProperties(css::uno::Reference<css::beans::XPropertySet>& xPropertySet, bool bLeft, - bool bTop, bool bRight, bool bBottom); + void setCellProperties(const css::uno::Reference<css::beans::XPropertySet>& xPropertySet, + bool bLeft, bool bTop, bool bRight, bool bBottom); public: DataTableView(rtl::Reference<::chart::ChartModel> const& xChartDoc, diff --git a/chart2/source/view/main/DataTableView.cxx b/chart2/source/view/main/DataTableView.cxx index 33f25afa77e0..6911e9cd5560 100644 --- a/chart2/source/view/main/DataTableView.cxx +++ b/chart2/source/view/main/DataTableView.cxx @@ -40,7 +40,7 @@ namespace chart { namespace { -void setTopCell(uno::Reference<beans::XPropertySet>& xPropertySet) +void setTopCell(const uno::Reference<beans::XPropertySet>& xPropertySet) { xPropertySet->setPropertyValue(u"FillColor"_ustr, uno::Any(Color(0xFFFFFF))); xPropertySet->setPropertyValue(u"TextVerticalAdjust"_ustr, @@ -55,8 +55,8 @@ void setTopCell(uno::Reference<beans::XPropertySet>& xPropertySet) xPropertySet->setPropertyValue(u"LeftBorder"_ustr, uno::Any(aBorderLine)); } -void copyProperty(uno::Reference<beans::XPropertySet>& xOut, - uno::Reference<beans::XPropertySet>& xIn, OUString const& sPropertyName) +void copyProperty(const uno::Reference<beans::XPropertySet>& xOut, + const uno::Reference<beans::XPropertySet>& xIn, OUString const& sPropertyName) { xOut->setPropertyValue(sPropertyName, xIn->getPropertyValue(sPropertyName)); } @@ -99,7 +99,7 @@ DataTableView::DataTableView( } void DataTableView::setCellCharAndParagraphProperties( - uno::Reference<beans::XPropertySet>& xPropertySet) + const uno::Reference<beans::XPropertySet>& xPropertySet) { uno::Reference<beans::XPropertySet> xDataTableProperties(m_xDataTableModel); @@ -156,7 +156,7 @@ void DataTableView::setCellCharAndParagraphProperties( xPropertySet->setPropertyValue(u"ParaAdjust"_ustr, uno::Any(style::ParagraphAdjust_CENTER)); } -void DataTableView::setCellProperties(css::uno::Reference<beans::XPropertySet>& xPropertySet, +void DataTableView::setCellProperties(const css::uno::Reference<beans::XPropertySet>& xPropertySet, bool bLeft, bool bTop, bool bRight, bool bBottom) { xPropertySet->setPropertyValue(u"FillColor"_ustr, uno::Any(Color(0xFFFFFF)));
