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>

19 files changed, 9 insertions, 63 deletions

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;