diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-08 10:30:36 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-09 07:44:49 +0200 |
| commit | 682fdbf1312cf6ca70fe209bf4d7051dad8f5008 (patch) | |
| tree | d001d22a20752086c6aa0f56d6c83791406fae9b /chart2 | |
| parent | c71896debc7ef5f6d6e918f59fa44423eb5df480 (diff) | |
loplugin:redundantpointerops check other pointer types
as well as unique_ptr
Change-Id: I54842bca161ee460fb96c46ca31b6f9c0a7dbbdf
Reviewed-on: https://gerrit.libreoffice.org/80455
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'chart2')
4 files changed, 5 insertions, 5 deletions
diff --git a/chart2/source/controller/main/ChartController_Insert.cxx b/chart2/source/controller/main/ChartController_Insert.cxx index 5789ebdd27c9..ec45a22eae0e 100644 --- a/chart2/source/controller/main/ChartController_Insert.cxx +++ b/chart2/source/controller/main/ChartController_Insert.cxx @@ -195,7 +195,7 @@ void ChartController::executeDispatch_DeleteLegend() ActionDescriptionProvider::ActionType::Delete, SchResId( STR_OBJECT_LEGEND )), m_xUndoManager ); - ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel().get()); + ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel()); LegendHelper::hideLegend(rModel); aUndoGuard.commit(); } @@ -207,7 +207,7 @@ void ChartController::executeDispatch_InsertLegend() ActionDescriptionProvider::ActionType::Insert, SchResId( STR_OBJECT_LEGEND )), m_xUndoManager ); - ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel().get()); + ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel()); Reference< chart2::XLegend > xLegend = LegendHelper::showLegend(rModel, m_xCC); aUndoGuard.commit(); } diff --git a/chart2/source/controller/main/ChartController_Position.cxx b/chart2/source/controller/main/ChartController_Position.cxx index f38bba593a15..410d67461118 100644 --- a/chart2/source/controller/main/ChartController_Position.cxx +++ b/chart2/source/controller/main/ChartController_Position.cxx @@ -192,7 +192,7 @@ void ChartController::executeDispatch_PositionAndSize(const ::css::uno::Sequence bool bChanged = false; if ( eObjectType == OBJECTTYPE_LEGEND ) { - ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel().get()); + ChartModel& rModel = dynamic_cast<ChartModel&>(*getModel()); bChanged = DiagramHelper::switchDiagramPositioningToExcludingPositioning(rModel, false , true); } diff --git a/chart2/source/controller/main/ChartController_Tools.cxx b/chart2/source/controller/main/ChartController_Tools.cxx index 423165322252..74f7e9ab6f79 100644 --- a/chart2/source/controller/main/ChartController_Tools.cxx +++ b/chart2/source/controller/main/ChartController_Tools.cxx @@ -998,7 +998,7 @@ void ChartController::impl_switchDiagramPositioningToExcludingPositioning() ActionDescriptionProvider::ActionType::PosSize, ObjectNameProvider::getName( OBJECTTYPE_DIAGRAM)), m_xUndoManager ); - ChartModel& rModel = dynamic_cast<ChartModel&>(*m_aModel->getModel().get()); + ChartModel& rModel = dynamic_cast<ChartModel&>(*m_aModel->getModel()); if (DiagramHelper::switchDiagramPositioningToExcludingPositioning(rModel, true, true)) aUndoGuard.commit(); } diff --git a/chart2/source/controller/main/ChartModelClone.cxx b/chart2/source/controller/main/ChartModelClone.cxx index 339547943552..456bd178c81b 100644 --- a/chart2/source/controller/main/ChartModelClone.cxx +++ b/chart2/source/controller/main/ChartModelClone.cxx @@ -192,7 +192,7 @@ namespace chart Reference< XChartDocument > xDestination( i_model, UNO_QUERY_THROW ); // propagate the correct flag for plotting of hidden values to the data provider and all used sequences - ChartModel& rModel = dynamic_cast<ChartModel&>(*i_model.get()); + ChartModel& rModel = dynamic_cast<ChartModel&>(*i_model); ChartModelHelper::setIncludeHiddenCells(ChartModelHelper::isIncludeHiddenCells( i_modelToCopyFrom ), rModel); // diagram |