diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-09 13:59:52 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-12 07:37:24 +0100 |
commit | 2f5868c4309d49ad495e0763b5b57d1f7f98e377 (patch) | |
tree | ba498138c67436ce30b5d9585353c50a1c5020bf /chart2 | |
parent | cb34e6083cdc82333d64ad9224e0d25e895d1dae (diff) |
loplugin:redundantfcast look for redundant copies in return statements
Change-Id: I5f416c865dfe1c36018784246a8007452eb42008
Reviewed-on: https://gerrit.libreoffice.org/50996
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'chart2')
-rw-r--r-- | chart2/source/controller/sidebar/ChartAreaPanel.cxx | 4 | ||||
-rw-r--r-- | chart2/source/model/template/DataInterpreter.cxx | 2 | ||||
-rw-r--r-- | chart2/source/tools/CachedDataSequence.cxx | 2 | ||||
-rw-r--r-- | chart2/source/tools/UncachedDataSequence.cxx | 2 | ||||
-rw-r--r-- | chart2/source/view/diagram/VDiagram.cxx | 4 |
5 files changed, 7 insertions, 7 deletions
diff --git a/chart2/source/controller/sidebar/ChartAreaPanel.cxx b/chart2/source/controller/sidebar/ChartAreaPanel.cxx index c057ebc5e0ad..0423aca7e8d1 100644 --- a/chart2/source/controller/sidebar/ChartAreaPanel.cxx +++ b/chart2/source/controller/sidebar/ChartAreaPanel.cxx @@ -112,7 +112,7 @@ XGradient getXGradientForName(const css::uno::Reference<css::frame::XModel>& xMo continue; if (pGradient->GetName() == rName) - return XGradient(pGradient->GetGradient()); + return pGradient->GetGradient(); } } catch (...) @@ -163,7 +163,7 @@ XHatch getXHatchFromName(const css::uno::Reference<css::frame::XModel>& xModel, { // we need to update the hatch name rName = pHatch->GetName(); - return XHatch(pHatch->GetHatch()); + return pHatch->GetHatch(); } } } diff --git a/chart2/source/model/template/DataInterpreter.cxx b/chart2/source/model/template/DataInterpreter.cxx index a29d49afd2c0..940287572b51 100644 --- a/chart2/source/model/template/DataInterpreter.cxx +++ b/chart2/source/model/template/DataInterpreter.cxx @@ -297,7 +297,7 @@ Reference< data::XDataSource > SAL_CALL DataInterpreter::mergeInterpretedData( } } - return Reference< data::XDataSource >( DataSourceHelper::createDataSource( comphelper::containerToSequence( aResultVec ) ) ); + return DataSourceHelper::createDataSource( comphelper::containerToSequence( aResultVec ) ); } // convenience methods diff --git a/chart2/source/tools/CachedDataSequence.cxx b/chart2/source/tools/CachedDataSequence.cxx index 72f2e544e79e..6c3024726585 100644 --- a/chart2/source/tools/CachedDataSequence.cxx +++ b/chart2/source/tools/CachedDataSequence.cxx @@ -226,7 +226,7 @@ IMPLEMENT_FORWARD_XTYPEPROVIDER2( CachedDataSequence, CachedDataSequence_Base, O // ____ XPropertySet ____ Reference< beans::XPropertySetInfo > SAL_CALL CachedDataSequence::getPropertySetInfo() { - return Reference< beans::XPropertySetInfo >( createPropertySetInfo( getInfoHelper() ) ); + return createPropertySetInfo( getInfoHelper() ); } // ____ ::comphelper::OPropertySetHelper ____ diff --git a/chart2/source/tools/UncachedDataSequence.cxx b/chart2/source/tools/UncachedDataSequence.cxx index 682500515416..6ace68e005de 100644 --- a/chart2/source/tools/UncachedDataSequence.cxx +++ b/chart2/source/tools/UncachedDataSequence.cxx @@ -125,7 +125,7 @@ IMPLEMENT_FORWARD_XTYPEPROVIDER2( UncachedDataSequence, UncachedDataSequence_Bas // ____ XPropertySet ____ Reference< beans::XPropertySetInfo > SAL_CALL UncachedDataSequence::getPropertySetInfo() { - return Reference< beans::XPropertySetInfo >( createPropertySetInfo( getInfoHelper() ) ); + return createPropertySetInfo( getInfoHelper() ); } // ____ ::comphelper::OPropertySetHelper ____ diff --git a/chart2/source/view/diagram/VDiagram.cxx b/chart2/source/view/diagram/VDiagram.cxx index 829cb3265260..4c5bac5abfc0 100644 --- a/chart2/source/view/diagram/VDiagram.cxx +++ b/chart2/source/view/diagram/VDiagram.cxx @@ -142,7 +142,7 @@ void VDiagram::createShapes( const awt::Point& rPos, const awt::Size& rSize ) m_xWall2D->setPosition(m_aCurrentPosWithoutAxes); } - return ::basegfx::B2IRectangle( BaseGFXHelper::makeRectangle(m_aCurrentPosWithoutAxes,m_aCurrentSizeWithoutAxes) ); + return BaseGFXHelper::makeRectangle(m_aCurrentPosWithoutAxes,m_aCurrentSizeWithoutAxes); } void VDiagram::createShapes_2d() @@ -462,7 +462,7 @@ void VDiagram::adjustAspectRatio3d( const awt::Size& rAvailableSize ) rPos, rAvailableSize, m_aCurrentSizeWithoutAxes ); m_xOuterGroupShape->setPosition(m_aCurrentPosWithoutAxes); - return ::basegfx::B2IRectangle( BaseGFXHelper::makeRectangle(m_aCurrentPosWithoutAxes,m_aCurrentSizeWithoutAxes) ); + return BaseGFXHelper::makeRectangle(m_aCurrentPosWithoutAxes,m_aCurrentSizeWithoutAxes); } void VDiagram::createShapes_3d() |