diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-17 09:32:21 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-22 11:16:54 +0200 |
commit | 51b49228ee7d3bb9788b7ff44e2c1737caa63818 (patch) | |
tree | 08a810a27a463537c86a56773adb1e32dc2f03f4 | |
parent | e129d1ae092e3605656ca4c58be3b77495444e5f (diff) |
loplugin:unusedfields in avmedia..comphelper
Change-Id: I74f125103b67c506d9bb67537e4c4cd3f8f871da
Reviewed-on: https://gerrit.libreoffice.org/54641
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
25 files changed, 14 insertions, 75 deletions
diff --git a/avmedia/source/vlc/vlcplayer.cxx b/avmedia/source/vlc/vlcplayer.cxx index ec109eaa23dd..a36064030921 100644 --- a/avmedia/source/vlc/vlcplayer.cxx +++ b/avmedia/source/vlc/vlcplayer.cxx @@ -43,9 +43,8 @@ VLCPlayer::VLCPlayer( const rtl::OUString& url, wrapper::Instance& instance, wrapper::EventHandler& eh ) : VLC_Base( m_aMutex ) - , mInstance( instance ) , mEventHandler( eh ) - , mMedia( url, mInstance ) + , mMedia( url, instance ) , mPlayer( mMedia ) , mEventManager( mPlayer, mEventHandler ) , mUrl( url ) diff --git a/avmedia/source/vlc/vlcplayer.hxx b/avmedia/source/vlc/vlcplayer.hxx index a3d555842862..8e5298175bc8 100644 --- a/avmedia/source/vlc/vlcplayer.hxx +++ b/avmedia/source/vlc/vlcplayer.hxx @@ -39,7 +39,6 @@ typedef ::cppu::WeakComponentImplHelper< css::media::XPlayer, class VLCPlayer : public ::cppu::BaseMutex, public VLC_Base { - wrapper::Instance& mInstance; wrapper::EventHandler& mEventHandler; wrapper::Media mMedia; diff --git a/basctl/source/dlged/dlged.cxx b/basctl/source/dlged/dlged.cxx index f3755414bfb3..5588752c4054 100644 --- a/basctl/source/dlged/dlged.cxx +++ b/basctl/source/dlged/dlged.cxx @@ -190,7 +190,6 @@ DlgEditor::DlgEditor ( ,eMode( DlgEditor::SELECT ) ,eActObj( OBJ_DLG_PUSHBUTTON ) ,bFirstDraw(false) - ,aGridSize( 100, 100 ) // 100TH_MM ,bCreateOK(true) ,bDialogModelChanged(false) ,aMarkIdle("basctl DlgEditor Mark") @@ -227,6 +226,7 @@ DlgEditor::DlgEditor ( pDlgEdView->SetMoveSnapOnlyTopLeft(true); pDlgEdView->SetWorkArea( tools::Rectangle( Point( 0, 0 ), pDlgEdPage->GetSize() ) ); + Size aGridSize( 100, 100 ); // 100TH_MM pDlgEdView->SetGridCoarse( aGridSize ); pDlgEdView->SetSnapGridWidth(Fraction(aGridSize.Width(), 1), Fraction(aGridSize.Height(), 1)); pDlgEdView->SetGridSnap( true ); diff --git a/basctl/source/inc/dlged.hxx b/basctl/source/inc/dlged.hxx index d844255c837b..6654b454a42d 100644 --- a/basctl/source/inc/dlged.hxx +++ b/basctl/source/inc/dlged.hxx @@ -126,7 +126,6 @@ private: Mode eMode; sal_uInt16 eActObj; bool bFirstDraw; - Size aGridSize; bool bCreateOK; tools::Rectangle aPaintRect; bool bDialogModelChanged; diff --git a/canvas/source/opengl/ogl_spritedevicehelper.cxx b/canvas/source/opengl/ogl_spritedevicehelper.cxx index bab170e37d24..08ab90564c68 100644 --- a/canvas/source/opengl/ogl_spritedevicehelper.cxx +++ b/canvas/source/opengl/ogl_spritedevicehelper.cxx @@ -515,7 +515,6 @@ namespace oglcanvas class BufferContextImpl : public IBufferContext { - ::basegfx::B2IVector maSize; GLuint mnFrambufferId; GLuint mnDepthId; GLuint mnTextureId; @@ -537,12 +536,11 @@ namespace oglcanvas public: explicit BufferContextImpl(const ::basegfx::B2IVector& rSize) : - maSize(rSize), mnFrambufferId(0), mnDepthId(0), mnTextureId(0) { - OpenGLHelper::createFramebuffer(maSize.getX(), maSize.getY(), mnFrambufferId, + OpenGLHelper::createFramebuffer(rSize.getX(), rSize.getY(), mnFrambufferId, mnDepthId, mnTextureId, false); } diff --git a/chart2/source/controller/dialogs/dlg_InsertDataLabel.cxx b/chart2/source/controller/dialogs/dlg_InsertDataLabel.cxx index 490ca57ed761..646ba40b8699 100644 --- a/chart2/source/controller/dialogs/dlg_InsertDataLabel.cxx +++ b/chart2/source/controller/dialogs/dlg_InsertDataLabel.cxx @@ -27,11 +27,10 @@ DataLabelsDialog::DataLabelsDialog(vcl::Window* pWindow, const SfxItemSet& rInAt ModalDialog(pWindow ,"dlg_DataLabels" ,"modules/schart/ui/dlg_DataLabel.ui"), - m_apDataLabelResources( new DataLabelResources(this, this, rInAttrs) ), - m_rInAttrs(rInAttrs) + m_apDataLabelResources( new DataLabelResources(this, this, rInAttrs) ) { m_apDataLabelResources->SetNumberFormatter( pFormatter ); - m_apDataLabelResources->Reset(m_rInAttrs); + m_apDataLabelResources->Reset(rInAttrs); } DataLabelsDialog::~DataLabelsDialog() = default; diff --git a/chart2/source/controller/dialogs/dlg_InsertErrorBars.cxx b/chart2/source/controller/dialogs/dlg_InsertErrorBars.cxx index a866ada4a8d6..e70a22f5e717 100644 --- a/chart2/source/controller/dialogs/dlg_InsertErrorBars.cxx +++ b/chart2/source/controller/dialogs/dlg_InsertErrorBars.cxx @@ -44,9 +44,8 @@ InsertErrorBarsDialog::InsertErrorBarsDialog( ModalDialog( pParent ,"dlg_InsertErrorBars" ,"modules/schart/ui/dlg_InsertErrorBars.ui"), - rInAttrs( rMyAttrs ), m_apErrorBarResources( new ErrorBarResources( - this, this, rInAttrs, + this, this, rMyAttrs, /* bNoneAvailable = */ true, eType )) { ObjectType objType = eType == ErrorBarResources::ERROR_BAR_Y ? OBJECTTYPE_DATA_ERRORS_Y : OBJECTTYPE_DATA_ERRORS_X; diff --git a/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx b/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx index 9d6df50719c6..f4a511aa52fd 100644 --- a/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx +++ b/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx @@ -325,7 +325,6 @@ SchAttribTabDlg::SchAttribTabDlg(vcl::Window* pParent, const uno::Reference< util::XNumberFormatsSupplier >& xNumberFormatsSupplier ) : SfxTabDialog(pParent, "AttributeDialog", "modules/schart/ui/attributedialog.ui", pAttr) - , eObjectType(pDialogParameter->getObjectType()) , nDlgType(nNoArrowNoShadowDlg) , m_pParameter( pDialogParameter ) , m_pViewElementListProvider( pViewElementListProvider ) @@ -342,7 +341,7 @@ SchAttribTabDlg::SchAttribTabDlg(vcl::Window* pParent, SvtCJKOptions aCJKOptions; - switch (eObjectType) + switch (pDialogParameter->getObjectType()) { case OBJECTTYPE_TITLE: AddTabPage(RID_SVXPAGE_LINE, SchResId(STR_PAGE_BORDER)); diff --git a/chart2/source/controller/dialogs/res_ErrorBar.cxx b/chart2/source/controller/dialogs/res_ErrorBar.cxx index 654cb794ddb2..6600108b8bf6 100644 --- a/chart2/source/controller/dialogs/res_ErrorBar.cxx +++ b/chart2/source/controller/dialogs/res_ErrorBar.cxx @@ -86,7 +86,6 @@ ErrorBarResources::ErrorBarResources( VclBuilderContainer* pParent, Dialog * pPa m_bIndicatorUnique( true ), m_bRangePosUnique( true ), m_bRangeNegUnique( true ), - m_bNoneAvailable( bNoneAvailable ), m_eErrorBarType( eType ), m_nConstDecimalDigits( 1 ), m_nConstSpinSize( 1 ), @@ -126,7 +125,7 @@ ErrorBarResources::ErrorBarResources( VclBuilderContainer* pParent, Dialog * pPa pParent->get(m_pUIStringNeg, "STR_DATA_SELECT_RANGE_FOR_NEGATIVE_ERRORBARS"); pParent->get(m_pUIStringRbRange, "STR_CONTROLTEXT_ERROR_BARS_FROM_DATA"); - if( m_bNoneAvailable ) + if( bNoneAvailable ) m_pRbNone->SetClickHdl( LINK( this, ErrorBarResources, CategoryChosen )); else m_pRbNone->Hide(); diff --git a/chart2/source/controller/inc/dlg_InsertDataLabel.hxx b/chart2/source/controller/inc/dlg_InsertDataLabel.hxx index 43a67490648f..c1660dc20fda 100644 --- a/chart2/source/controller/inc/dlg_InsertDataLabel.hxx +++ b/chart2/source/controller/inc/dlg_InsertDataLabel.hxx @@ -38,8 +38,6 @@ private: // HelpButton m_aBtnHelp; std::unique_ptr < DataLabelResources > m_apDataLabelResources; - const SfxItemSet& m_rInAttrs; - public: DataLabelsDialog(vcl::Window* pParent, const SfxItemSet& rInAttrs, SvNumberFormatter* pFormatter); virtual ~DataLabelsDialog() override; diff --git a/chart2/source/controller/inc/dlg_InsertErrorBars.hxx b/chart2/source/controller/inc/dlg_InsertErrorBars.hxx index 584033a45584..796b0dc46888 100644 --- a/chart2/source/controller/inc/dlg_InsertErrorBars.hxx +++ b/chart2/source/controller/inc/dlg_InsertErrorBars.hxx @@ -48,8 +48,6 @@ public: virtual void DataChanged( const DataChangedEvent& rDCEvt ) override; private: - const SfxItemSet & rInAttrs; - std::unique_ptr< ErrorBarResources > m_apErrorBarResources; }; diff --git a/chart2/source/controller/inc/dlg_ObjectProperties.hxx b/chart2/source/controller/inc/dlg_ObjectProperties.hxx index d8702f689191..afa83bc6f56c 100644 --- a/chart2/source/controller/inc/dlg_ObjectProperties.hxx +++ b/chart2/source/controller/inc/dlg_ObjectProperties.hxx @@ -106,7 +106,6 @@ class ViewElementListProvider; class SchAttribTabDlg : public SfxTabDialog { private: - ObjectType eObjectType; sal_uInt16 nDlgType; const ObjectPropertiesDialogParameter * const m_pParameter; diff --git a/chart2/source/controller/inc/res_ErrorBar.hxx b/chart2/source/controller/inc/res_ErrorBar.hxx index e5ade413f9a4..a2688c288da4 100644 --- a/chart2/source/controller/inc/res_ErrorBar.hxx +++ b/chart2/source/controller/inc/res_ErrorBar.hxx @@ -106,8 +106,6 @@ private: bool m_bRangePosUnique; bool m_bRangeNegUnique; - bool m_bNoneAvailable; - tErrorBarType m_eErrorBarType; sal_uInt16 m_nConstDecimalDigits; sal_Int64 m_nConstSpinSize; diff --git a/chart2/source/inc/MediaDescriptorHelper.hxx b/chart2/source/inc/MediaDescriptorHelper.hxx index be9d258d9ae0..73e1b3fe4c91 100644 --- a/chart2/source/inc/MediaDescriptorHelper.hxx +++ b/chart2/source/inc/MediaDescriptorHelper.hxx @@ -58,15 +58,10 @@ public: css::uno::Sequence< css::beans::PropertyValue > m_aDeprecatedProperties; //these are properties which are described in service com.sun.star.document.MediaDescriptor but are marked as deprecated - css::uno::Sequence< css::beans::PropertyValue > - m_aAdditionalProperties; //these are properties which are not described in service com.sun.star.document.MediaDescriptor - //properties which should be given to a model are additionally stored in this sequence (not documented properties and deprecated properties are not included!) css::uno::Sequence< css::beans::PropertyValue > m_aModelProperties; //these are properties which are not described in service com.sun.star.document.MediaDescriptor - css::uno::Any ComponentData; - css::uno::Any FilterData; OUString FilterName; //internal filter name. bool ISSET_FilterName; @@ -84,8 +79,6 @@ public: OUString URL;// FileName, URL of the document. bool ISSET_URL; - css::uno::Any ViewData; - // new framework objects css::uno::Reference< css::embed::XStorage > Storage; diff --git a/chart2/source/model/inc/BaseCoordinateSystem.hxx b/chart2/source/model/inc/BaseCoordinateSystem.hxx index 06955837a955..76ad85aa597f 100644 --- a/chart2/source/model/inc/BaseCoordinateSystem.hxx +++ b/chart2/source/model/inc/BaseCoordinateSystem.hxx @@ -119,7 +119,6 @@ private: sal_Int32 m_nDimensionCount; typedef std::vector< std::vector< css::uno::Reference< css::chart2::XAxis > > > tAxisVecVecType; tAxisVecVecType m_aAllAxis; //outer sequence is the dimension; inner sequence is the axis index that indicates main or secondary axis - css::uno::Sequence< css::uno::Any > m_aOrigin; std::vector< css::uno::Reference< css::chart2::XChartType > > m_aChartTypes; }; diff --git a/chart2/source/model/inc/StockBar.hxx b/chart2/source/model/inc/StockBar.hxx index ed3afcf20a99..b1247e584a63 100644 --- a/chart2/source/model/inc/StockBar.hxx +++ b/chart2/source/model/inc/StockBar.hxx @@ -85,7 +85,6 @@ private: virtual void firePropertyChangeEvent() override; using OPropertySet::disposing; - const bool m_bRisingCourse; css::uno::Reference< css::util::XModifyListener > m_xModifyEventForwarder; }; diff --git a/chart2/source/model/main/BaseCoordinateSystem.cxx b/chart2/source/model/main/BaseCoordinateSystem.cxx index 04782bde3757..dabba0889268 100644 --- a/chart2/source/model/main/BaseCoordinateSystem.cxx +++ b/chart2/source/model/main/BaseCoordinateSystem.cxx @@ -144,10 +144,6 @@ BaseCoordinateSystem::BaseCoordinateSystem( xAxis->setScaleData( aScaleData ); } - m_aOrigin.realloc( m_nDimensionCount ); - for( sal_Int32 i = 0; i < m_nDimensionCount; ++i ) - m_aOrigin[ i ] <<= 0.0; - setFastPropertyValue_NoBroadcast( PROP_COORDINATESYSTEM_SWAPXANDYAXIS, uno::Any( false )); } @@ -158,8 +154,7 @@ BaseCoordinateSystem::BaseCoordinateSystem( MutexContainer(), ::property::OPropertySet( rSource, m_aMutex ), m_xModifyEventForwarder( ModifyListenerHelper::createModifyEventForwarder()), - m_nDimensionCount( rSource.m_nDimensionCount ), - m_aOrigin( rSource.m_aOrigin ) + m_nDimensionCount( rSource.m_nDimensionCount ) { m_aAllAxis.resize(rSource.m_aAllAxis.size()); tAxisVecVecType::size_type nN=0; diff --git a/chart2/source/model/main/StockBar.cxx b/chart2/source/model/main/StockBar.cxx index a9b2ff2cc8a2..2f7a534b23ce 100644 --- a/chart2/source/model/main/StockBar.cxx +++ b/chart2/source/model/main/StockBar.cxx @@ -111,10 +111,9 @@ namespace chart StockBar::StockBar( bool bRisingCourse ) : ::property::OPropertySet( m_aMutex ), - m_bRisingCourse( bRisingCourse ), m_xModifyEventForwarder( ModifyListenerHelper::createModifyEventForwarder()) { - if( ! m_bRisingCourse ) + if( ! bRisingCourse ) { setFastPropertyValue_NoBroadcast( ::chart::FillProperties::PROP_FILL_COLOR, @@ -129,7 +128,6 @@ StockBar::StockBar( const StockBar & rOther ) : MutexContainer(), impl::StockBar_Base(), ::property::OPropertySet( rOther, m_aMutex ), - m_bRisingCourse( rOther.m_bRisingCourse ), m_xModifyEventForwarder( ModifyListenerHelper::createModifyEventForwarder()) {} diff --git a/chart2/source/tools/MediaDescriptorHelper.cxx b/chart2/source/tools/MediaDescriptorHelper.cxx index d95ddf18bc44..7ae409976dd6 100644 --- a/chart2/source/tools/MediaDescriptorHelper.cxx +++ b/chart2/source/tools/MediaDescriptorHelper.cxx @@ -28,13 +28,11 @@ MediaDescriptorHelper::MediaDescriptorHelper( const uno::Sequence< beans::PropertyValue > & rMediaDescriptor ) : m_aRegularProperties(rMediaDescriptor.getLength()) , m_aDeprecatedProperties(rMediaDescriptor.getLength()) - , m_aAdditionalProperties(rMediaDescriptor.getLength()) , m_aModelProperties(rMediaDescriptor.getLength()) { impl_init(); sal_Int32 nRegularCount = 0; sal_Int32 nDeprecatedCount = 0; - sal_Int32 nAdditionalCount = 0; sal_Int32 nModelCount = 0; auto addRegularProp = [this, &nRegularCount](const beans::PropertyValue& rRegularProp) @@ -77,7 +75,6 @@ MediaDescriptorHelper::MediaDescriptorHelper( const uno::Sequence< } else if (rProp.Name == "ComponentData") { - ComponentData = rProp.Value; addModelProp(rProp); } else if (rProp.Name == "FileName") @@ -86,7 +83,6 @@ MediaDescriptorHelper::MediaDescriptorHelper( const uno::Sequence< } else if (rProp.Name == "FilterData") { - FilterData = rProp.Value; addModelProp(rProp); } else if (rProp.Name == "FilterName") @@ -223,7 +219,6 @@ MediaDescriptorHelper::MediaDescriptorHelper( const uno::Sequence< } else if (rProp.Name == "ViewData") { - ViewData = rProp.Value; addModelProp(rProp); } else if (rProp.Name == "ViewId") @@ -234,16 +229,10 @@ MediaDescriptorHelper::MediaDescriptorHelper( const uno::Sequence< { addDepreciatedProp(rProp); } - else - { - m_aAdditionalProperties[nAdditionalCount]=rProp; - nAdditionalCount++; - } } m_aRegularProperties.realloc(nRegularCount); m_aDeprecatedProperties.realloc(nDeprecatedCount); - m_aAdditionalProperties.realloc(nAdditionalCount); m_aModelProperties.realloc(nModelCount); } diff --git a/chart2/source/view/axes/Tickmarks_Dates.cxx b/chart2/source/view/axes/Tickmarks_Dates.cxx index d1daac59ebfe..b9442374a737 100644 --- a/chart2/source/view/axes/Tickmarks_Dates.cxx +++ b/chart2/source/view/axes/Tickmarks_Dates.cxx @@ -44,14 +44,6 @@ DateTickFactory::DateTickFactory( m_xInverseScaling = m_aScale.Scaling->getInverseScaling(); OSL_ENSURE( m_xInverseScaling.is(), "each Scaling needs to return a inverse Scaling" ); } - - m_fScaledVisibleMin = m_aScale.Minimum; - if( m_xInverseScaling.is() ) - m_fScaledVisibleMin = m_aScale.Scaling->doScaling(m_fScaledVisibleMin); - - m_fScaledVisibleMax = m_aScale.Maximum; - if( m_xInverseScaling.is() ) - m_fScaledVisibleMax = m_aScale.Scaling->doScaling(m_fScaledVisibleMax); } DateTickFactory::~DateTickFactory() diff --git a/chart2/source/view/axes/Tickmarks_Dates.hxx b/chart2/source/view/axes/Tickmarks_Dates.hxx index ac121d02924b..2e4e3b35b58a 100644 --- a/chart2/source/view/axes/Tickmarks_Dates.hxx +++ b/chart2/source/view/axes/Tickmarks_Dates.hxx @@ -43,10 +43,6 @@ private: //member ExplicitIncrementData m_aIncrement; css::uno::Reference< css::chart2::XScaling > m_xInverseScaling; - - //minimum and maximum of the visible range after scaling - double m_fScaledVisibleMin; - double m_fScaledVisibleMax; }; } //namespace chart diff --git a/chart2/source/view/charttypes/GL3DBarChart.cxx b/chart2/source/view/charttypes/GL3DBarChart.cxx index 86d4fb3e6d5c..60309d08ad2a 100644 --- a/chart2/source/view/charttypes/GL3DBarChart.cxx +++ b/chart2/source/view/charttypes/GL3DBarChart.cxx @@ -82,16 +82,14 @@ double findMaxValue(const std::vector<std::unique_ptr<VDataSeries> >& rDataSerie class SharedResourceAccess { private: - osl::Condition& mrCond1; osl::Condition& mrCond2; public: SharedResourceAccess(osl::Condition& rCond1, osl::Condition& rCond2): - mrCond1(rCond1), mrCond2(rCond2) { - mrCond1.set(); + rCond1.set(); } ~SharedResourceAccess() diff --git a/chart2/source/view/main/DummyXShape.cxx b/chart2/source/view/main/DummyXShape.cxx index 71cd47206b2d..035353dbd795 100644 --- a/chart2/source/view/main/DummyXShape.cxx +++ b/chart2/source/view/main/DummyXShape.cxx @@ -758,7 +758,6 @@ private: DummyText::DummyText(const OUString& rText, const tNameSequence& rNames, const tAnySequence& rValues, const uno::Any& rTrans, uno::Reference< drawing::XShapes > const & xTarget, double nRotation ): - maText(rText), maTrans(rTrans), mnRotation(nRotation) { @@ -768,7 +767,7 @@ DummyText::DummyText(const OUString& rText, const tNameSequence& rNames, DummyChart* pChart = getRootShape(); TextCache& rCache = pChart->getTextCache(); TextCache::TextCacheKey aKey; - aKey.maText = maText; + aKey.maText = rText; aKey.maProperties = maProperties; int bmpWidth; int bmpHeight; diff --git a/chart2/source/view/main/DummyXShape.hxx b/chart2/source/view/main/DummyXShape.hxx index bd15c8450523..30e29239c0d4 100644 --- a/chart2/source/view/main/DummyXShape.hxx +++ b/chart2/source/view/main/DummyXShape.hxx @@ -314,7 +314,6 @@ public: private: void setTransformatAsProperty(const css::drawing::HomogenMatrix3& rMatrix); - OUString maText; css::uno::Any maTrans; BitmapEx maBitmap; double mnRotation; diff --git a/comphelper/source/misc/logging.cxx b/comphelper/source/misc/logging.cxx index e08d9d540a99..55bfefef4bbe 100644 --- a/comphelper/source/misc/logging.cxx +++ b/comphelper/source/misc/logging.cxx @@ -41,7 +41,6 @@ namespace comphelper { private: Reference< XComponentContext > m_aContext; - OUString m_sLoggerName; Reference< XLogger > m_xLogger; public: @@ -53,13 +52,12 @@ namespace comphelper EventLogger_Impl::EventLogger_Impl( const Reference< XComponentContext >& _rxContext, const OUString& _rLoggerName ) :m_aContext( _rxContext ) - ,m_sLoggerName( _rLoggerName ) { try { Reference< XLoggerPool > xPool( LoggerPool::get( m_aContext ) ); - if ( !m_sLoggerName.isEmpty() ) - m_xLogger = xPool->getNamedLogger( m_sLoggerName ); + if ( !_rLoggerName.isEmpty() ) + m_xLogger = xPool->getNamedLogger( _rLoggerName ); else m_xLogger = xPool->getDefaultLogger(); } |