loplugin:constparams in chart2

Change-Id: Ic325b79f04e04aa19e08a60db30b982d90f04c80
Reviewed-on: https://gerrit.libreoffice.org/40480
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

22 files changed, 57 insertions, 57 deletions

diff --git a/chart2/source/view/axes/VCartesianAxis.cxx b/chart2/source/view/axes/VCartesianAxis.cxx
index 6b355b3734b4..63d50e40e172 100644
--- a/chart2/source/view/axes/VCartesianAxis.cxx
+++ b/chart2/source/view/axes/VCartesianAxis.cxx
@@ -666,7 +666,7 @@ sal_Int32 VCartesianAxis::getTextLevelCount() const
 
 bool VCartesianAxis::createTextShapes(
     const Reference<drawing::XShapes>& xTarget, TickIter& rTickIter,
-    AxisLabelProperties& rAxisLabelProperties, TickFactory2D* pTickFactory,
+    AxisLabelProperties& rAxisLabelProperties, TickFactory2D const * pTickFactory,
     sal_Int32 nScreenDistanceBetweenTicks )
 {
     const bool bIsHorizontalAxis = pTickFactory->isHorizontalAxis();
@@ -881,7 +881,7 @@ bool VCartesianAxis::createTextShapes(
 
 bool VCartesianAxis::createTextShapesSimple(
     const Reference<drawing::XShapes>& xTarget, TickIter& rTickIter,
-    AxisLabelProperties& rAxisLabelProperties, TickFactory2D* pTickFactory )
+    AxisLabelProperties& rAxisLabelProperties, TickFactory2D const * pTickFactory )
 {
     FixedNumberFormatter aFixedNumberFormatter(
                 m_xNumberFormatsSupplier, rAxisLabelProperties.nNumberFormatKey );
@@ -1511,7 +1511,7 @@ sal_Int32 VCartesianAxis::estimateMaximumAutoMainIncrementCount()
     return nRet;
 }
 
-void VCartesianAxis::doStaggeringOfLabels( const AxisLabelProperties& rAxisLabelProperties, TickFactory2D* pTickFactory2D )
+void VCartesianAxis::doStaggeringOfLabels( const AxisLabelProperties& rAxisLabelProperties, TickFactory2D const * pTickFactory2D )
 {
     if( !pTickFactory2D )
         return;
@@ -1716,7 +1716,7 @@ void VCartesianAxis::updatePositions()
     doStaggeringOfLabels( m_aAxisLabelProperties, pTickFactory2D );
 }
 
-void VCartesianAxis::createTickMarkLineShapes( TickInfoArrayType& rTickInfos, const TickmarkProperties& rTickmarkProperties, TickFactory2D& rTickFactory2D, bool bOnlyAtLabels )
+void VCartesianAxis::createTickMarkLineShapes( TickInfoArrayType& rTickInfos, const TickmarkProperties& rTickmarkProperties, TickFactory2D const & rTickFactory2D, bool bOnlyAtLabels )
 {
     sal_Int32 nPointCount = rTickInfos.size();
     drawing::PointSequenceSequence aPoints(2*nPointCount);
diff --git a/chart2/source/view/axes/VCartesianAxis.hxx b/chart2/source/view/axes/VCartesianAxis.hxx
index 15074393d90a..df15e4896198 100644
--- a/chart2/source/view/axes/VCartesianAxis.hxx
+++ b/chart2/source/view/axes/VCartesianAxis.hxx
@@ -120,7 +120,7 @@ private: //methods
     bool createTextShapes(
         const css::uno::Reference<css::drawing::XShapes >& xTarget,
         TickIter& rTickIter, AxisLabelProperties& rAxisLabelProperties,
-        TickFactory2D* pTickFactory, sal_Int32 nScreenDistanceBetweenTicks );
+        TickFactory2D const * pTickFactory, sal_Int32 nScreenDistanceBetweenTicks );
 
     /**
      * Variant of createTextShapes where none of auto-staggering and
@@ -130,9 +130,9 @@ private: //methods
     bool createTextShapesSimple(
         const css::uno::Reference<css::drawing::XShapes >& xTarget,
         TickIter& rTickIter, AxisLabelProperties& rAxisLabelProperties,
-        TickFactory2D* pTickFactory );
+        TickFactory2D const * pTickFactory );
 
-    void createTickMarkLineShapes( TickInfoArrayType& rTickInfos, const TickmarkProperties& rTickmarkProperties, TickFactory2D& rTickFactory2D, bool bOnlyAtLabels );
+    void createTickMarkLineShapes( TickInfoArrayType& rTickInfos, const TickmarkProperties& rTickmarkProperties, TickFactory2D const & rTickFactory2D, bool bOnlyAtLabels );
 
     TickFactory2D* createTickFactory2D();
     void hideIdenticalScreenValues( TickInfoArraysType& rTickInfos ) const;
@@ -144,7 +144,7 @@ private: //methods
      * positions based on the final stagger setting.
      */
     void    doStaggeringOfLabels( const AxisLabelProperties& rAxisLabelProperties
-                            , TickFactory2D* pTickFactory2D );
+                            , TickFactory2D const * pTickFactory2D );
 
     /**
      * @return true if we can break a single line label text into multiple
diff --git a/chart2/source/view/axes/VPolarGrid.cxx b/chart2/source/view/axes/VPolarGrid.cxx
index 54e8f7b357a4..279f38c956f2 100644
--- a/chart2/source/view/axes/VPolarGrid.cxx
+++ b/chart2/source/view/axes/VPolarGrid.cxx
@@ -67,7 +67,7 @@ void VPolarGrid::createLinePointSequence_ForAngleAxis(
         , TickInfoArraysType& rAllTickInfos
         , const ExplicitIncrementData& rIncrement
         , const ExplicitScaleData& rScale
-        , PolarPlottingPositionHelper* pPosHelper
+        , PolarPlottingPositionHelper const * pPosHelper
         , double fLogicRadius, double fLogicZ )
 {
     Reference< XScaling > xInverseScaling( nullptr );
diff --git a/chart2/source/view/axes/VPolarGrid.hxx b/chart2/source/view/axes/VPolarGrid.hxx
index f7c177071567..4391d7eca335 100644
--- a/chart2/source/view/axes/VPolarGrid.hxx
+++ b/chart2/source/view/axes/VPolarGrid.hxx
@@ -49,7 +49,7 @@ public:
                     , TickInfoArraysType& rAllTickInfos
                     , const ExplicitIncrementData& rIncrement
                     , const ExplicitScaleData& rScale
-                    , PolarPlottingPositionHelper* pPosHelper
+                    , PolarPlottingPositionHelper const * pPosHelper
                     , double fLogicRadius, double fLogicZ );
 
 private: //member
diff --git a/chart2/source/view/charttypes/AreaChart.cxx b/chart2/source/view/charttypes/AreaChart.cxx
index 2bedb8a64142..9ef01a23fbb2 100644
--- a/chart2/source/view/charttypes/AreaChart.cxx
+++ b/chart2/source/view/charttypes/AreaChart.cxx
@@ -240,7 +240,7 @@ void lcl_removeDuplicatePoints( drawing::PolyPolygonShape3D& rPolyPoly, Plotting
     rPolyPoly=aTmp;
 }
 
-bool AreaChart::create_stepped_line( drawing::PolyPolygonShape3D aStartPoly, chart2::CurveStyle eCurveStyle, PlottingPositionHelper* pPosHelper, drawing::PolyPolygonShape3D &aPoly )
+bool AreaChart::create_stepped_line( drawing::PolyPolygonShape3D aStartPoly, chart2::CurveStyle eCurveStyle, PlottingPositionHelper const * pPosHelper, drawing::PolyPolygonShape3D &aPoly )
 {
     sal_uInt32 nOuterCount = aStartPoly.SequenceX.getLength();
     if ( !nOuterCount )
@@ -366,7 +366,7 @@ bool AreaChart::create_stepped_line( drawing::PolyPolygonShape3D aStartPoly, cha
 }
 
 bool AreaChart::impl_createLine( VDataSeries* pSeries
-                , drawing::PolyPolygonShape3D* pSeriesPoly
+                , drawing::PolyPolygonShape3D const * pSeriesPoly
                 , PlottingPositionHelper* pPosHelper )
 {
     //return true if a line was created successfully
@@ -452,9 +452,9 @@ bool AreaChart::impl_createLine( VDataSeries* pSeries
 }
 
 bool AreaChart::impl_createArea( VDataSeries* pSeries
-                , drawing::PolyPolygonShape3D* pSeriesPoly
-                , drawing::PolyPolygonShape3D* pPreviousSeriesPoly
-                , PlottingPositionHelper* pPosHelper )
+                , drawing::PolyPolygonShape3D const * pSeriesPoly
+                , drawing::PolyPolygonShape3D const * pPreviousSeriesPoly
+                , PlottingPositionHelper const * pPosHelper )
 {
     //return true if an area was created successfully
 
diff --git a/chart2/source/view/charttypes/AreaChart.hxx b/chart2/source/view/charttypes/AreaChart.hxx
index af8c4251c5b9..acc133667668 100644
--- a/chart2/source/view/charttypes/AreaChart.hxx
+++ b/chart2/source/view/charttypes/AreaChart.hxx
@@ -54,15 +54,15 @@ public:
 private: //methods
     void impl_createSeriesShapes();
     bool impl_createArea( VDataSeries* pSeries
-                , css::drawing::PolyPolygonShape3D* pSeriesPoly
-                , css::drawing::PolyPolygonShape3D* pPreviousSeriesPoly
-                , PlottingPositionHelper* pPosHelper );
+                , css::drawing::PolyPolygonShape3D const * pSeriesPoly
+                , css::drawing::PolyPolygonShape3D const * pPreviousSeriesPoly
+                , PlottingPositionHelper const * pPosHelper );
     bool impl_createLine( VDataSeries* pSeries
-                , css::drawing::PolyPolygonShape3D* pSeriesPoly
+                , css::drawing::PolyPolygonShape3D const * pSeriesPoly
                 , PlottingPositionHelper* pPosHelper );
     static bool create_stepped_line( css::drawing::PolyPolygonShape3D aStartPoly
                 , css::chart2::CurveStyle eCurveStyle
-                , PlottingPositionHelper* pPosHelper
+                , PlottingPositionHelper const * pPosHelper
                 , css::drawing::PolyPolygonShape3D &aPoly );
 
 private: //member
diff --git a/chart2/source/view/charttypes/BarChart.cxx b/chart2/source/view/charttypes/BarChart.cxx
index 08b9d5a6c6bb..702b90f7569c 100644
--- a/chart2/source/view/charttypes/BarChart.cxx
+++ b/chart2/source/view/charttypes/BarChart.cxx
@@ -133,7 +133,7 @@ awt::Point BarChart::getLabelScreenPositionAndAlignment(
                      LabelAlignment& rAlignment, sal_Int32 nLabelPlacement
                      , double fScaledX, double fScaledLowerYValue, double fScaledUpperYValue, double fScaledZ
                      , double fScaledLowerBarDepth, double fScaledUpperBarDepth, double fBaseValue
-                     , BarPositionHelper* pPosHelper
+                     , BarPositionHelper const * pPosHelper
                      ) const
 {
     double fX = fScaledX;
diff --git a/chart2/source/view/charttypes/BarChart.hxx b/chart2/source/view/charttypes/BarChart.hxx
index d276fa531eab..da6071802a90 100644
--- a/chart2/source/view/charttypes/BarChart.hxx
+++ b/chart2/source/view/charttypes/BarChart.hxx
@@ -56,7 +56,7 @@ private: //methods
                         LabelAlignment& rAlignment, sal_Int32 nLabelPlacement
                         , double fScaledX, double fScaledLowerYValue, double fScaledUpperYValue, double fScaledZ
                         , double fScaledLowerBarDepth, double fScaledUpperBarDepth, double fBaseValue
-                        , BarPositionHelper* pPosHelper ) const;
+                        , BarPositionHelper const * pPosHelper ) const;
 
     virtual PlottingPositionHelper& getPlottingPositionHelper( sal_Int32 nAxisIndex ) const override;//nAxisIndex indicates whether the position belongs to the main axis ( nAxisIndex==0 ) or secondary axis ( nAxisIndex==1 )
 
diff --git a/chart2/source/view/charttypes/GL3DBarChart.cxx b/chart2/source/view/charttypes/GL3DBarChart.cxx
index 1eecec9f67f9..c0085bdf49df 100644
--- a/chart2/source/view/charttypes/GL3DBarChart.cxx
+++ b/chart2/source/view/charttypes/GL3DBarChart.cxx
@@ -1168,7 +1168,7 @@ void GL3DBarChart::updateRenderFPS()
                        glm::vec4(1.0f, 0.0f, 0.0f, 0.0f), glm::vec3(0.0f, 0.0f, 0.0f));
 }
 
-int GL3DBarChart::calcTimeInterval(TimeValue &startTime, TimeValue &endTime)
+int GL3DBarChart::calcTimeInterval(TimeValue const &startTime, TimeValue const &endTime)
 {
     TimeValue aTime;
     aTime.Seconds = endTime.Seconds - startTime.Seconds - 1;
@@ -1215,7 +1215,7 @@ void GL3DBarChart::updateDataUpdateFPS()
     addScreenTextShape(maDataUpdateFPS, glm::vec2(-0.77f, 0.92f), 0.07f, true, glm::vec4(1.0f, 0.0f, 0.0f, 0.0f));
 }
 
-void GL3DBarChart::recordBarHistory(sal_uInt32 &nBarID, float &nVal)
+void GL3DBarChart::recordBarHistory(sal_uInt32 nBarID, float nVal)
 {
     std::deque<float>& aList = maBarHistory[nBarID];
     if(aList.size() == HISTORY_NUM)
diff --git a/chart2/source/view/charttypes/NetChart.cxx b/chart2/source/view/charttypes/NetChart.cxx
index ba896e3eb4e8..4ed5410c0729 100644
--- a/chart2/source/view/charttypes/NetChart.cxx
+++ b/chart2/source/view/charttypes/NetChart.cxx
@@ -118,7 +118,7 @@ drawing::Direction3D NetChart::getPreferredDiagramAspectRatio() const
 
 bool NetChart::impl_createLine( VDataSeries* pSeries
                 , drawing::PolyPolygonShape3D* pSeriesPoly
-                , PlottingPositionHelper* pPosHelper )
+                , PlottingPositionHelper const * pPosHelper )
 {
     //return true if a line was created successfully
     uno::Reference< drawing::XShapes > xSeriesGroupShape_Shapes = getSeriesGroupShapeBackChild(pSeries, m_xSeriesTarget);
@@ -171,8 +171,8 @@ bool NetChart::impl_createLine( VDataSeries* pSeries
 
 bool NetChart::impl_createArea( VDataSeries* pSeries
                 , drawing::PolyPolygonShape3D* pSeriesPoly
-                , drawing::PolyPolygonShape3D* pPreviousSeriesPoly
-                , PlottingPositionHelper* pPosHelper )
+                , drawing::PolyPolygonShape3D const * pPreviousSeriesPoly
+                , PlottingPositionHelper const * pPosHelper )
 {
     //return true if an area was created successfully
 
diff --git a/chart2/source/view/charttypes/NetChart.hxx b/chart2/source/view/charttypes/NetChart.hxx
index 626e9253097c..eecd2550b25a 100644
--- a/chart2/source/view/charttypes/NetChart.hxx
+++ b/chart2/source/view/charttypes/NetChart.hxx
@@ -55,11 +55,11 @@ private: //methods
     void impl_createSeriesShapes();
     bool impl_createArea( VDataSeries* pSeries
                 , css::drawing::PolyPolygonShape3D* pSeriesPoly
-                , css::drawing::PolyPolygonShape3D* pPreviousSeriesPoly
-                , PlottingPositionHelper* pPosHelper );
+                , css::drawing::PolyPolygonShape3D const * pPreviousSeriesPoly
+                , PlottingPositionHelper const * pPosHelper );
     bool impl_createLine( VDataSeries* pSeries
                 , css::drawing::PolyPolygonShape3D* pSeriesPoly
-                , PlottingPositionHelper* pPosHelper );
+                , PlottingPositionHelper const * pPosHelper );
 
 private: //member
     std::unique_ptr<PlottingPositionHelper> m_pMainPosHelper;
diff --git a/chart2/source/view/charttypes/PieChart.cxx b/chart2/source/view/charttypes/PieChart.cxx
index 72df0caf6566..84ad1938459d 100644
--- a/chart2/source/view/charttypes/PieChart.cxx
+++ b/chart2/source/view/charttypes/PieChart.cxx
@@ -217,7 +217,7 @@ bool PieChart::shouldSnapRectToUsedArea()
 uno::Reference< drawing::XShape > PieChart::createDataPoint(
     const uno::Reference<drawing::XShapes>& xTarget,
     const uno::Reference<beans::XPropertySet>& xObjectProperties,
-    tPropertyNameValueMap* pOverwritePropertiesMap,
+    tPropertyNameValueMap const * pOverwritePropertiesMap,
     const ShapeParam& rParam )
 {
     //transform position:
@@ -995,7 +995,7 @@ bool PieChart::detectLabelOverlapsAndMove( const awt::Size& rPageSize )
 /** Try to remove all overlaps that occur in the list of labels going from
  *  `pFirstBorder` to `pSecondBorder`
  */
-bool PieChart::tryMoveLabels( PieLabelInfo* pFirstBorder, PieLabelInfo* pSecondBorder
+bool PieChart::tryMoveLabels( PieLabelInfo const * pFirstBorder, PieLabelInfo const * pSecondBorder
                              , PieLabelInfo* pCenter
                              , bool bSingleCenter, bool& rbAlternativeMoveDirection, const awt::Size& rPageSize )
 {
@@ -1263,7 +1263,7 @@ void PieChart::rearrangeLabelToAvoidOverlapIfRequested( const awt::Size& rPageSi
  *   4. the top edge when 225 < alpha < 315.
  *
  **/
-bool PieChart::performLabelBestFitInnerPlacement(ShapeParam& rShapeParam, PieLabelInfo& rPieLabelInfo)
+bool PieChart::performLabelBestFitInnerPlacement(ShapeParam& rShapeParam, PieLabelInfo const & rPieLabelInfo)
 {
     SAL_INFO( "chart2.pie.label.bestfit.inside",
               "** PieChart::performLabelBestFitInnerPlacement invoked **" );
diff --git a/chart2/source/view/charttypes/PieChart.hxx b/chart2/source/view/charttypes/PieChart.hxx
index dd6c5f2de600..1b39ba229ed0 100644
--- a/chart2/source/view/charttypes/PieChart.hxx
+++ b/chart2/source/view/charttypes/PieChart.hxx
@@ -68,7 +68,7 @@ private: //methods
         createDataPoint(
             const css::uno::Reference<css::drawing::XShapes>& xTarget,
             const css::uno::Reference<css::beans::XPropertySet>& xObjectProperties,
-            tPropertyNameValueMap* pOverWritePropertiesMap,
+            tPropertyNameValueMap const * pOverWritePropertiesMap,
             const ShapeParam& rParam );
 
     /** This method creates a text shape for a label of a data point.
@@ -102,11 +102,11 @@ private: //methods
     bool                detectLabelOverlapsAndMove(const css::awt::Size& rPageSize);//returns true when there might be more to do
     void                resetLabelPositionsToPreviousState();
 struct PieLabelInfo;
-    bool                tryMoveLabels( PieLabelInfo* pFirstBorder, PieLabelInfo* pSecondBorder
+    bool                tryMoveLabels( PieLabelInfo const * pFirstBorder, PieLabelInfo const * pSecondBorder
                                 , PieLabelInfo* pCenter, bool bSingleCenter, bool& rbAlternativeMoveDirection
                                 , const css::awt::Size& rPageSize );
 
-    bool                performLabelBestFitInnerPlacement(ShapeParam& rShapeParam, PieLabelInfo& rPieLabelInfo);
+    bool                performLabelBestFitInnerPlacement(ShapeParam& rShapeParam, PieLabelInfo const & rPieLabelInfo);
     static bool         performLabelBestFitOuterPlacement(ShapeParam& rShapeParam, PieLabelInfo& rPieLabelInfo);
     void                performLabelBestFit(ShapeParam& rShapeParam, PieLabelInfo& rPieLabelInfo);
 
diff --git a/chart2/source/view/charttypes/VSeriesPlotter.cxx b/chart2/source/view/charttypes/VSeriesPlotter.cxx
index b4173b69fe41..589812594ff6 100644
--- a/chart2/source/view/charttypes/VSeriesPlotter.cxx
+++ b/chart2/source/view/charttypes/VSeriesPlotter.cxx
@@ -357,7 +357,7 @@ uno::Reference< drawing::XShapes > VSeriesPlotter::getErrorBarsGroupShape( VData
 
 }
 
-OUString VSeriesPlotter::getLabelTextForValue( VDataSeries& rDataSeries
+OUString VSeriesPlotter::getLabelTextForValue( VDataSeries const & rDataSeries
                 , sal_Int32 nPointIndex
                 , double fValue
                 , bool bAsPercentage )
@@ -802,7 +802,7 @@ void lcl_AddErrorBottomLine( const drawing::Position3D& rPosition, ::basegfx::B2
 ::basegfx::B2DVector lcl_getErrorBarMainDirection(
               const drawing::Position3D& rStart
             , const drawing::Position3D& rBottomEnd
-            , PlottingPositionHelper* pPosHelper
+            , PlottingPositionHelper const * pPosHelper
             , const drawing::Position3D& rUnscaledLogicPosition
             , bool bYError )
 {
@@ -843,7 +843,7 @@ void lcl_AddErrorBottomLine( const drawing::Position3D& rPosition, ::basegfx::B2
     return aMainDirection;
 }
 
-drawing::Position3D lcl_transformMixedToScene( PlottingPositionHelper* pPosHelper
+drawing::Position3D lcl_transformMixedToScene( PlottingPositionHelper const * pPosHelper
     , double fX /*scaled*/, double fY /*unscaled*/, double fZ /*unscaled*/, bool bClip )
 {
     if(!pPosHelper)
@@ -1024,7 +1024,7 @@ void VSeriesPlotter::createErrorBar_X( const drawing::Position3D& rUnscaledLogic
 void VSeriesPlotter::createErrorBar_Y( const drawing::Position3D& rUnscaledLogicPosition
                             , VDataSeries& rVDataSeries, sal_Int32 nPointIndex
                             , const uno::Reference< drawing::XShapes >& xTarget
-                            , double* pfScaledLogicX )
+                            , double const * pfScaledLogicX )
 {
     if(m_nDimension!=2)
         return;
@@ -1043,7 +1043,7 @@ void VSeriesPlotter::createErrorBar_Y( const drawing::Position3D& rUnscaledLogic
     }
 }
 
-void VSeriesPlotter::createRegressionCurvesShapes( VDataSeries& rVDataSeries,
+void VSeriesPlotter::createRegressionCurvesShapes( VDataSeries const & rVDataSeries,
                             const uno::Reference< drawing::XShapes >& xTarget,
                             const uno::Reference< drawing::XShapes >& xEquationTarget,
                             bool bMaySkipPoints )
@@ -1192,7 +1192,7 @@ void VSeriesPlotter::createRegressionCurvesShapes( VDataSeries& rVDataSeries,
     }
 }
 
-sal_Int32 lcl_getOUStringMaxLineLength ( OUStringBuffer& aString )
+sal_Int32 lcl_getOUStringMaxLineLength ( OUStringBuffer const & aString )
 {
     const sal_Int32 nStringLength = aString.getLength();
     sal_Int32 nMaxLineLength = 0;
@@ -1357,7 +1357,7 @@ void VSeriesPlotter::setMappedProperties(
           const uno::Reference< drawing::XShape >& xTargetShape
         , const uno::Reference< beans::XPropertySet >& xSource
         , const tPropertyNameMap& rMap
-        , tPropertyNameValueMap* pOverwriteMap )
+        , tPropertyNameValueMap const * pOverwriteMap )
 {
     uno::Reference< beans::XPropertySet > xTargetProp( xTargetShape, uno::UNO_QUERY );
     PropertyMapper::setMappedProperties(xTargetProp,xSource,rMap,pOverwriteMap);
diff --git a/chart2/source/view/inc/GL3DBarChart.hxx b/chart2/source/view/inc/GL3DBarChart.hxx
index 8e83508b560f..753504ccf17a 100644
--- a/chart2/source/view/inc/GL3DBarChart.hxx
+++ b/chart2/source/view/inc/GL3DBarChart.hxx
@@ -102,12 +102,12 @@ private:
     void updateRenderFPS();
     void updateDataUpdateFPS();
     DECL_LINK(UpdateTimerHdl, Timer*, void);
-    static int calcTimeInterval(TimeValue &startTime, TimeValue &endTime);
+    static int calcTimeInterval(TimeValue const &startTime, TimeValue const &endTime);
     float addScreenTextShape(OUString &nStr, const glm::vec2& rLeftOrRightTop, float nTextHeight, bool bLeftTopFlag,
                                   const glm::vec4& rColor,
                                   const glm::vec3& rPos = glm::vec3(0.0f, 0.0f, 0.0f),
                                   sal_uInt32 nEvent = 0);
-    void recordBarHistory(sal_uInt32 &nBarID, float &nVal);
+    void recordBarHistory(sal_uInt32 nBarID, float nVal);
     void updateClickEvent();
     void calcDistance(std::vector<sal_uInt32> &vectorNearest);
     static float calcScrollDistance(const glm::mat4 &mvp, const glm::vec3& rPos);
diff --git a/chart2/source/view/inc/GL3DRenderer.hxx b/chart2/source/view/inc/GL3DRenderer.hxx
index 240fb955bb17..3a3f19a78bbf 100644
--- a/chart2/source/view/inc/GL3DRenderer.hxx
+++ b/chart2/source/view/inc/GL3DRenderer.hxx
@@ -248,11 +248,11 @@ private:
     void AddNormalData(GLuint normalBuf);
     void AddIndexData(GLuint indexBuf);
     void RenderNonRoundedBar(const Extrude3DInfo& extrude3D);
-    static bool GetSimilarVertexIndex(PackedVertex & packed,
+    static bool GetSimilarVertexIndex(PackedVertex const & packed,
         std::map<PackedVertex,unsigned short> & VertexToOutIndex,
         unsigned short & result
         );
-    static void SetVertex(PackedVertex &packed,
+    static void SetVertex(PackedVertex const &packed,
         std::map<PackedVertex,unsigned short> &VertexToOutIndex,
         std::vector<glm::vec3> &vertex,
         std::vector<glm::vec3> &normal,
diff --git a/chart2/source/view/inc/PropertyMapper.hxx b/chart2/source/view/inc/PropertyMapper.hxx
index 5af3b7fdf27b..6d2b1508fd7b 100644
--- a/chart2/source/view/inc/PropertyMapper.hxx
+++ b/chart2/source/view/inc/PropertyMapper.hxx
@@ -49,7 +49,7 @@ public:
           const css::uno::Reference< css::beans::XPropertySet >& xTarget
         , const css::uno::Reference< css::beans::XPropertySet >& xSource
         , const tPropertyNameMap& rMap
-        , tPropertyNameValueMap* pOverwriteMap=nullptr );
+        , tPropertyNameValueMap const * pOverwriteMap=nullptr );
 
     /**
      * Fetch property values from the source object and map it to the
diff --git a/chart2/source/view/inc/VSeriesPlotter.hxx b/chart2/source/view/inc/VSeriesPlotter.hxx
index c66f992cf12a..65aa67de86b0 100644
--- a/chart2/source/view/inc/VSeriesPlotter.hxx
+++ b/chart2/source/view/inc/VSeriesPlotter.hxx
@@ -327,7 +327,7 @@ protected:
 
     /// This method returns a text string representation of the passed numeric
     /// value by exploiting a NumberFormatterWrapper object.
-    OUString getLabelTextForValue( VDataSeries& rDataSeries
+    OUString getLabelTextForValue( VDataSeries const & rDataSeries
                 , sal_Int32 nPointIndex
                 , double fValue
                 , bool bAsPercentage );
@@ -366,9 +366,9 @@ protected:
     void createErrorBar_Y( const css::drawing::Position3D& rUnscaledLogicPosition
         , VDataSeries& rVDataSeries, sal_Int32 nPointIndex
         , const css::uno::Reference< css::drawing::XShapes >& xTarget
-        , double* pfScaledLogicX );
+        , double const * pfScaledLogicX );
 
-    void createRegressionCurvesShapes( VDataSeries& rVDataSeries
+    void createRegressionCurvesShapes( VDataSeries const & rVDataSeries
         , const css::uno::Reference< css::drawing::XShapes >& xTarget
         , const css::uno::Reference< css::drawing::XShapes >& xEquationTarget
         , bool bMaySkipPointsInRegressionCalculation );
@@ -383,7 +383,7 @@ protected:
           const css::uno::Reference< css::drawing::XShape >& xTarget
         , const css::uno::Reference< css::beans::XPropertySet >& xSource
         , const tPropertyNameMap& rMap
-        , tPropertyNameValueMap* pOverwriteMap=nullptr );
+        , tPropertyNameValueMap const * pOverwriteMap=nullptr );
 
     virtual PlottingPositionHelper& getPlottingPositionHelper( sal_Int32 nAxisIndex ) const;//nAxisIndex indicates whether the position belongs to the main axis ( nAxisIndex==0 ) or secondary axis ( nAxisIndex==1 )
 
diff --git a/chart2/source/view/main/ChartView.cxx b/chart2/source/view/main/ChartView.cxx
index 20a4c1cb493c..965157e13fa9 100644
--- a/chart2/source/view/main/ChartView.cxx
+++ b/chart2/source/view/main/ChartView.cxx
@@ -2296,7 +2296,7 @@ bool getAvailablePosAndSizeForDiagram(
 enum TitleAlignment { ALIGN_LEFT, ALIGN_TOP, ALIGN_RIGHT, ALIGN_BOTTOM, ALIGN_Z };
 
 void changePositionOfAxisTitle( VTitle* pVTitle, TitleAlignment eAlignment
-                               , awt::Rectangle& rDiagramPlusAxesRect, const awt::Size & rPageSize )
+                               , awt::Rectangle const & rDiagramPlusAxesRect, const awt::Size & rPageSize )
 {
     if(!pVTitle)
         return;
diff --git a/chart2/source/view/main/DrawModelWrapper.cxx b/chart2/source/view/main/DrawModelWrapper.cxx
index d0dd9234aa16..580a762ecb2a 100644
--- a/chart2/source/view/main/DrawModelWrapper.cxx
+++ b/chart2/source/view/main/DrawModelWrapper.cxx
@@ -297,7 +297,7 @@ SdrObject* DrawModelWrapper::getNamedSdrObject( const OUString& rName )
     return getNamedSdrObject( rName, GetPage(0) );
 }
 
-SdrObject* DrawModelWrapper::getNamedSdrObject( const OUString& rObjectCID, SdrObjList* pSearchList )
+SdrObject* DrawModelWrapper::getNamedSdrObject( const OUString& rObjectCID, SdrObjList const * pSearchList )
 {
     if(!pSearchList || rObjectCID.isEmpty())
         return nullptr;
diff --git a/chart2/source/view/main/GL3DRenderer.cxx b/chart2/source/view/main/GL3DRenderer.cxx
index 8c92ccd9a9b2..3ff7939aba2f 100644
--- a/chart2/source/view/main/GL3DRenderer.cxx
+++ b/chart2/source/view/main/GL3DRenderer.cxx
@@ -468,7 +468,7 @@ void OpenGL3DRenderer::AddIndexData(GLuint indexBuf)
     glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, 0);
 }
 
-bool OpenGL3DRenderer::GetSimilarVertexIndex(PackedVertex & packed,
+bool OpenGL3DRenderer::GetSimilarVertexIndex(PackedVertex const & packed,
     std::map<PackedVertex,unsigned short> & VertexToOutIndex,
     unsigned short & result
     )
@@ -485,7 +485,7 @@ bool OpenGL3DRenderer::GetSimilarVertexIndex(PackedVertex & packed,
     }
 }
 
-void OpenGL3DRenderer::SetVertex(PackedVertex &packed,
+void OpenGL3DRenderer::SetVertex(PackedVertex const &packed,
     std::map<PackedVertex,unsigned short> &VertexToOutIndex,
     std::vector<glm::vec3> &vertex,
     std::vector<glm::vec3> &normal,
diff --git a/chart2/source/view/main/PropertyMapper.cxx b/chart2/source/view/main/PropertyMapper.cxx
index e35bd913129e..15bc0dbea5a6 100644
--- a/chart2/source/view/main/PropertyMapper.cxx
+++ b/chart2/source/view/main/PropertyMapper.cxx
@@ -51,7 +51,7 @@ void PropertyMapper::setMappedProperties(
           const uno::Reference< beans::XPropertySet >& xTarget
         , const uno::Reference< beans::XPropertySet >& xSource
         , const tPropertyNameMap& rMap
-        , tPropertyNameValueMap* pOverwriteMap )
+        , tPropertyNameValueMap const * pOverwriteMap )
 {
     if( !xTarget.is() || !xSource.is() )
         return;