include/linguistic/lngprophelp.hxx                           |   10 --
 l10ntools/inc/xmlparse.hxx                                   |    5 -
 l10ntools/source/xmlparse.cxx                                |   10 --
 linguistic/source/lngprophelp.cxx                            |   47 +++++------
 reportdesign/source/filter/xml/xmlStyleImport.cxx            |    2 
 reportdesign/source/filter/xml/xmlStyleImport.hxx            |    1 
 reportdesign/source/filter/xml/xmlfilter.cxx                 |    1 
 reportdesign/source/filter/xml/xmlfilter.hxx                 |    1 
 sax/source/expatwrap/sax_expat.cxx                           |    6 -
 sax/source/fastparser/fastparser.cxx                         |   24 -----
 sc/inc/externalrefmgr.hxx                                    |    3 
 sc/inc/mtvelements.hxx                                       |    1 
 sc/inc/pagepar.hxx                                           |    2 
 sc/source/core/data/column3.cxx                              |    1 
 sc/source/core/data/pagepar.cxx                              |    2 
 sc/source/filter/xml/xmldpimp.hxx                            |    4 
 sc/source/filter/xml/xmlfilti.cxx                            |    4 
 sc/source/filter/xml/xmltabi.cxx                             |    1 
 sc/source/filter/xml/xmltabi.hxx                             |    1 
 sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx |    4 
 sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx         |    1 
 sc/source/ui/docshell/externalrefmgr.cxx                     |    6 -
 sc/source/ui/inc/AccessibleSpreadsheet.hxx                   |    1 
 sc/source/ui/inc/preview.hxx                                 |    1 
 sc/source/ui/miscdlgs/dataproviderdlg.cxx                    |    7 -
 sc/source/ui/vba/vbaformatcondition.cxx                      |    1 
 sc/source/ui/vba/vbaformatcondition.hxx                      |    1 
 sc/source/ui/view/preview.cxx                                |    1 
 sc/source/ui/view/printfun.cxx                               |    2 
 scripting/source/vbaevents/eventhelper.cxx                   |   10 --
 30 files changed, 36 insertions(+), 125 deletions(-)

New commits:
commit c9611c5e6465948de029e9c2fbd17e75ee07d31f
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Fri Feb 22 15:31:15 2019 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sat Feb 23 07:30:12 2019 +0100

    loplugin:unusedfields in l10ntools..sax
    
    Change-Id: I6e4ff4853c3995072b020af54549dc81303a4fde
    Reviewed-on: https://gerrit.libreoffice.org/68225
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/include/linguistic/lngprophelp.hxx 
b/include/linguistic/lngprophelp.hxx
index f8613deb454e..f86d810e159c 100644
--- a/include/linguistic/lngprophelp.hxx
+++ b/include/linguistic/lngprophelp.hxx
@@ -27,6 +27,7 @@
 #include <com/sun/star/linguistic2/XLinguProperties.hpp>
 #include <com/sun/star/linguistic2/XLinguServiceEventBroadcaster.hpp>
 #include <linguistic/lngdllapi.h>
+#include <rtl/ref.hxx>
 
 namespace com { namespace sun { namespace star { namespace beans {
     class   XPropertySet;
@@ -148,8 +149,7 @@ public:
 
 class LNG_DLLPUBLIC PropertyHelper_Thesaurus
 {
-    PropertyHelper_Thes* pInst;
-    css::uno::Reference< css::beans::XPropertyChangeListener >    xPropHelper;
+    rtl::Reference< PropertyHelper_Thes >    mxPropHelper;
 
     // disallow use of copy-constructor and assignment-operator
     PropertyHelper_Thesaurus( const PropertyHelper_Thes & ) = delete;
@@ -209,8 +209,7 @@ public:
 
 class LNG_DLLPUBLIC PropertyHelper_Spelling
 {
-    PropertyHelper_Spell* pInst;
-    css::uno::Reference< css::beans::XPropertyChangeListener >    xPropHelper;
+    rtl::Reference< PropertyHelper_Spell >    mxPropHelper;
 
     // disallow use of copy-constructor and assignment-operator
     PropertyHelper_Spelling( const PropertyHelper_Spell & ) = delete;
@@ -279,8 +278,7 @@ public:
 
 class LNG_DLLPUBLIC PropertyHelper_Hyphenation
 {
-    PropertyHelper_Hyphen* pInst;
-    css::uno::Reference< css::beans::XPropertyChangeListener >    xPropHelper;
+    rtl::Reference< PropertyHelper_Hyphen >    mxPropHelper;
 
     // disallow use of copy-constructor and assignment-operator
     PropertyHelper_Hyphenation( const PropertyHelper_Hyphen & ) = delete;
diff --git a/l10ntools/inc/xmlparse.hxx b/l10ntools/inc/xmlparse.hxx
index 67b40ac2205c..a9d2893d68b3 100644
--- a/l10ntools/inc/xmlparse.hxx
+++ b/l10ntools/inc/xmlparse.hxx
@@ -206,8 +206,6 @@ class XMLElement : public XMLParentNode
 private:
     OString m_sElementName;
     std::unique_ptr<XMLAttributeList> m_pAttributes;
-    OString m_sId;
-    OString m_sLanguageId;
 
 protected:
     void Print(XMLNode *pCur, OStringBuffer& rBuffer, bool bRootelement) const;
@@ -237,9 +235,6 @@ public:
 
     /// Return a Unicode String representation of this object
     OString ToOString();
-
-    void SetId              ( OString const & sTheId )      { m_sId = sTheId; }
-    void SetLanguageId      ( OString const & sLangId )     { m_sLanguageId = 
sLangId; }
 };
 
 /** Holds character data
diff --git a/l10ntools/source/xmlparse.cxx b/l10ntools/source/xmlparse.cxx
index 369d8e6e1562..b6df209ed896 100644
--- a/l10ntools/source/xmlparse.cxx
+++ b/l10ntools/source/xmlparse.cxx
@@ -472,8 +472,6 @@ void XMLFile::SearchL10NElements( XMLChildNode *pCur )
                             sLanguage=(*pElement->GetAttributeList())[ j 
]->GetValue();
                         }
                     }
-                    pElement->SetLanguageId( sLanguage );
-                    pElement->SetId( sTmpStrVal );
                 }
 
                 if ( bInsert && ( m_aNodes_localize.find( sName ) != 
m_aNodes_localize.end() ) )
@@ -555,16 +553,12 @@ XMLElement::XMLElement(
 )
     : XMLParentNode( pParent )
     , m_sElementName( rName )
-    , m_sId(OString())
-    , m_sLanguageId(OString())
 {
 }
 
 XMLElement::XMLElement(const XMLElement& rObj)
     : XMLParentNode( rObj )
     , m_sElementName( rObj.m_sElementName )
-    , m_sId( rObj.m_sId )
-    , m_sLanguageId( rObj.m_sLanguageId )
 {
     if ( rObj.m_pAttributes )
     {
@@ -580,8 +574,6 @@ XMLElement& XMLElement::operator=(const XMLElement& rObj)
     {
         XMLParentNode::operator=(rObj);
         m_sElementName = rObj.m_sElementName;
-        m_sId = rObj.m_sId;
-        m_sLanguageId = rObj.m_sLanguageId;
 
         if ( m_pAttributes )
         {
@@ -608,7 +600,6 @@ void XMLElement::AddAttribute( const OString &rAttribute, 
const OString &rValue
 
 void XMLElement::ChangeLanguageTag( const OString &rValue )
 {
-    SetLanguageId(rValue);
     if ( m_pAttributes )
     {
         bool bWasSet = false;
@@ -635,7 +626,6 @@ void XMLElement::ChangeLanguageTag( const OString &rValue )
             {
                 XMLElement* pElem = static_cast< XMLElement* >(pNode);
                 pElem->ChangeLanguageTag( rValue );
-                pElem->SetLanguageId(rValue);
                 pElem  = nullptr;
                 pNode  = nullptr;
             }
diff --git a/linguistic/source/lngprophelp.cxx 
b/linguistic/source/lngprophelp.cxx
index 7c061be2a621..d1368b1e3a62 100644
--- a/linguistic/source/lngprophelp.cxx
+++ b/linguistic/source/lngprophelp.cxx
@@ -654,8 +654,7 @@ PropertyHelper_Thesaurus::PropertyHelper_Thesaurus(
             const css::uno::Reference< css::uno::XInterface > &rxSource,
             css::uno::Reference< css::linguistic2::XLinguProperties > const 
&rxPropSet )
 {
-    pInst = new PropertyHelper_Thes( rxSource, rxPropSet );
-    xPropHelper = pInst;
+    mxPropHelper = new PropertyHelper_Thes( rxSource, rxPropSet );
 }
 
 PropertyHelper_Thesaurus::~PropertyHelper_Thesaurus()
@@ -664,25 +663,24 @@ PropertyHelper_Thesaurus::~PropertyHelper_Thesaurus()
 
 void PropertyHelper_Thesaurus::AddAsPropListener()
 {
-    pInst->AddAsPropListener();
+    mxPropHelper->AddAsPropListener();
 }
 
 void PropertyHelper_Thesaurus::RemoveAsPropListener()
 {
-    pInst->RemoveAsPropListener();
+    mxPropHelper->RemoveAsPropListener();
 }
 
 void PropertyHelper_Thesaurus::SetTmpPropVals( const 
css::beans::PropertyValues &rPropVals )
 {
-    pInst->SetTmpPropVals( rPropVals );
+    mxPropHelper->SetTmpPropVals( rPropVals );
 }
 
 PropertyHelper_Hyphenation::PropertyHelper_Hyphenation(
             const css::uno::Reference< css::uno::XInterface > &rxSource,
             css::uno::Reference< css::linguistic2::XLinguProperties > const 
&rxPropSet)
 {
-    pInst = new PropertyHelper_Hyphen( rxSource, rxPropSet );
-    xPropHelper = pInst;
+    mxPropHelper = new PropertyHelper_Hyphen( rxSource, rxPropSet );
 }
 
 PropertyHelper_Hyphenation::~PropertyHelper_Hyphenation()
@@ -691,52 +689,51 @@ PropertyHelper_Hyphenation::~PropertyHelper_Hyphenation()
 
 void PropertyHelper_Hyphenation::AddAsPropListener()
 {
-    pInst->AddAsPropListener();
+    mxPropHelper->AddAsPropListener();
 }
 
 void PropertyHelper_Hyphenation::RemoveAsPropListener()
 {
-    pInst->RemoveAsPropListener();
+    mxPropHelper->RemoveAsPropListener();
 }
 
 void PropertyHelper_Hyphenation::SetTmpPropVals( const 
css::beans::PropertyValues &rPropVals )
 {
-    pInst->SetTmpPropVals( rPropVals );
+    mxPropHelper->SetTmpPropVals( rPropVals );
 }
 
 sal_Int16 PropertyHelper_Hyphenation::GetMinLeading() const
 {
-    return pInst->GetMinLeading();
+    return mxPropHelper->GetMinLeading();
 }
 
 sal_Int16 PropertyHelper_Hyphenation::GetMinTrailing() const
 {
-    return pInst->GetMinTrailing();
+    return mxPropHelper->GetMinTrailing();
 }
 
 sal_Int16 PropertyHelper_Hyphenation::GetMinWordLength() const
 {
-    return pInst->GetMinWordLength();
+    return mxPropHelper->GetMinWordLength();
 }
 
 bool PropertyHelper_Hyphenation::addLinguServiceEventListener(
                 const css::uno::Reference< 
css::linguistic2::XLinguServiceEventListener >& rxListener )
 {
-    return pInst->addLinguServiceEventListener( rxListener );
+    return mxPropHelper->addLinguServiceEventListener( rxListener );
 }
 
 bool PropertyHelper_Hyphenation::removeLinguServiceEventListener(
                 const css::uno::Reference< 
css::linguistic2::XLinguServiceEventListener >& rxListener )
 {
-    return pInst->removeLinguServiceEventListener( rxListener );
+    return mxPropHelper->removeLinguServiceEventListener( rxListener );
 }
 
 PropertyHelper_Spelling::PropertyHelper_Spelling(
             const css::uno::Reference< css::uno::XInterface > &rxSource,
             css::uno::Reference< css::linguistic2::XLinguProperties > const 
&rxPropSet )
 {
-    pInst = new PropertyHelper_Spell( rxSource, rxPropSet );
-    xPropHelper = pInst;
+    mxPropHelper = new PropertyHelper_Spell( rxSource, rxPropSet );
 }
 
 PropertyHelper_Spelling::~PropertyHelper_Spelling()
@@ -745,46 +742,46 @@ PropertyHelper_Spelling::~PropertyHelper_Spelling()
 
 void PropertyHelper_Spelling::AddAsPropListener()
 {
-    pInst->AddAsPropListener();
+    mxPropHelper->AddAsPropListener();
 }
 
 void PropertyHelper_Spelling::RemoveAsPropListener()
 {
-    pInst->RemoveAsPropListener();
+    mxPropHelper->RemoveAsPropListener();
 }
 
 void PropertyHelper_Spelling::SetTmpPropVals( const css::beans::PropertyValues 
&rPropVals )
 {
-    pInst->SetTmpPropVals( rPropVals );
+    mxPropHelper->SetTmpPropVals( rPropVals );
 }
 
 bool PropertyHelper_Spelling::IsSpellUpperCase() const
 {
-    return pInst->IsSpellUpperCase();
+    return mxPropHelper->IsSpellUpperCase();
 }
 
 bool PropertyHelper_Spelling::IsSpellWithDigits() const
 {
-    return pInst->IsSpellWithDigits();
+    return mxPropHelper->IsSpellWithDigits();
 }
 
 bool PropertyHelper_Spelling::IsSpellCapitalization() const
 {
-    return pInst->IsSpellCapitalization();
+    return mxPropHelper->IsSpellCapitalization();
 }
 
 bool PropertyHelper_Spelling::addLinguServiceEventListener(
                 const css::uno::Reference<
                     css::linguistic2::XLinguServiceEventListener >& rxListener 
)
 {
-    return pInst->addLinguServiceEventListener( rxListener );
+    return mxPropHelper->addLinguServiceEventListener( rxListener );
 }
 
 bool PropertyHelper_Spelling::removeLinguServiceEventListener(
                 const css::uno::Reference<
                     css::linguistic2::XLinguServiceEventListener >& rxListener 
)
 {
-    return pInst->removeLinguServiceEventListener( rxListener );
+    return mxPropHelper->removeLinguServiceEventListener( rxListener );
 }
 
 }   // namespace linguistic
diff --git a/reportdesign/source/filter/xml/xmlStyleImport.cxx 
b/reportdesign/source/filter/xml/xmlStyleImport.cxx
index 5d0f9b1383a7..704641996661 100644
--- a/reportdesign/source/filter/xml/xmlStyleImport.cxx
+++ b/reportdesign/source/filter/xml/xmlStyleImport.cxx
@@ -136,7 +136,7 @@ void OControlStyleContext::SetAttribute( sal_uInt16 
nPrefixKey,
     if( IsXMLToken(rLocalName, XML_DATA_STYLE_NAME ) )
         m_sDataStyleName = rValue;
     else if ( IsXMLToken(rLocalName, XML_MASTER_PAGE_NAME ) )
-        sPageStyle = rValue;
+        ;
     else
         XMLPropStyleContext::SetAttribute( nPrefixKey, rLocalName, rValue );
 }
diff --git a/reportdesign/source/filter/xml/xmlStyleImport.hxx 
b/reportdesign/source/filter/xml/xmlStyleImport.hxx
index 8ae7335f0006..5a59b8dab229 100644
--- a/reportdesign/source/filter/xml/xmlStyleImport.hxx
+++ b/reportdesign/source/filter/xml/xmlStyleImport.hxx
@@ -39,7 +39,6 @@ namespace rptxml
     class OControlStyleContext : public XMLPropStyleContext
     {
         OUString             m_sDataStyleName;
-        OUString             sPageStyle;
         SvXMLStylesContext*         pStyles;
         //  std::vector<ScXMLMapContent>    aMaps;
         sal_Int32                   m_nNumberFormat;
diff --git a/reportdesign/source/filter/xml/xmlfilter.cxx 
b/reportdesign/source/filter/xml/xmlfilter.cxx
index 2ff95dd1b3f8..1f54bf8f66c8 100644
--- a/reportdesign/source/filter/xml/xmlfilter.cxx
+++ b/reportdesign/source/filter/xml/xmlfilter.cxx
@@ -350,7 +350,6 @@ ORptFilter::ORptFilter( const uno::Reference< 
XComponentContext >& _rxContext, S
     m_xCellStylesPropertySetMapper = OXMLHelper::GetCellStylePropertyMap(true, 
false);
     m_xColumnStylesPropertySetMapper = new 
XMLPropertySetMapper(OXMLHelper::GetColumnStyleProps(), m_xPropHdlFactory, 
false);
     m_xRowStylesPropertySetMapper = new 
XMLPropertySetMapper(OXMLHelper::GetRowStyleProps(), m_xPropHdlFactory, false);
-    m_xTableStylesPropertySetMapper = new XMLTextPropertySetMapper( 
TextPropMap::TABLE_DEFAULTS, false );
 }
 
 
diff --git a/reportdesign/source/filter/xml/xmlfilter.hxx 
b/reportdesign/source/filter/xml/xmlfilter.hxx
index 079ef969cc7f..266198f46bfb 100644
--- a/reportdesign/source/filter/xml/xmlfilter.hxx
+++ b/reportdesign/source/filter/xml/xmlfilter.hxx
@@ -85,7 +85,6 @@ private:
     rtl::Reference < XMLPropertySetMapper >           
m_xCellStylesPropertySetMapper;
     rtl::Reference < XMLPropertySetMapper >           
m_xColumnStylesPropertySetMapper;
     rtl::Reference < XMLPropertySetMapper >           
m_xRowStylesPropertySetMapper;
-    rtl::Reference < XMLPropertySetMapper >           
m_xTableStylesPropertySetMapper;
 
     Reference<XReportDefinition>                    m_xReportDefinition;
     std::shared_ptr<rptui::OReportModel>        m_pReportModel;
diff --git a/sax/source/expatwrap/sax_expat.cxx 
b/sax/source/expatwrap/sax_expat.cxx
index 57437b093442..e98da4dd2a83 100644
--- a/sax/source/expatwrap/sax_expat.cxx
+++ b/sax/source/expatwrap/sax_expat.cxx
@@ -176,8 +176,6 @@ public: // module scope
     bool                bExceptionWasThrown;
     bool                bRTExceptionWasThrown;
 
-    Locale              locale;
-
 public:
     SaxExpatParser_Impl()
         : m_bEnableDoS(false)
@@ -512,9 +510,9 @@ void SaxExpatParser::setEntityResolver(const 
css::uno::Reference < XEntityResolv
 }
 
 
-void SaxExpatParser::setLocale( const Locale & locale )
+void SaxExpatParser::setLocale( const Locale & )
 {
-    m_pImpl->locale = locale;
+    // not implemented
 }
 
 // XServiceInfo
diff --git a/sax/source/fastparser/fastparser.cxx 
b/sax/source/fastparser/fastparser.cxx
index a8c0c8fc357c..1e8b578e325f 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -124,9 +124,7 @@ struct ParserData
     css::uno::Reference< css::xml::sax::XFastTokenHandler >    mxTokenHandler;
     FastTokenHandlerBase*                                      mpTokenHandler;
     css::uno::Reference< css::xml::sax::XErrorHandler >        mxErrorHandler;
-    css::uno::Reference< css::xml::sax::XEntityResolver >      
mxEntityResolver;
     css::uno::Reference< css::xml::sax::XFastNamespaceHandler 
>mxNamespaceHandler;
-    css::lang::Locale                                          maLocale;
 
     ParserData();
 };
@@ -228,11 +226,7 @@ public:
     /// @throws css::uno::RuntimeException
     void setErrorHandler( const css::uno::Reference< 
css::xml::sax::XErrorHandler >& Handler );
     /// @throws css::uno::RuntimeException
-    void setEntityResolver( const css::uno::Reference< 
css::xml::sax::XEntityResolver >& Resolver );
-    /// @throws css::uno::RuntimeException
     void setNamespaceHandler( const css::uno::Reference< 
css::xml::sax::XFastNamespaceHandler >& Handler);
-    /// @throws css::uno::RuntimeException
-    void setLocale( const css::lang::Locale& rLocale );
 
     // called by the C callbacks of the expat parser
     void callbackStartElement( const xmlChar *localName , const xmlChar* 
prefix, const xmlChar* URI,
@@ -917,16 +911,6 @@ void FastSaxParserImpl::setErrorHandler(const Reference< 
XErrorHandler > & Handl
     maData.mxErrorHandler = Handler;
 }
 
-void FastSaxParserImpl::setEntityResolver(const Reference < XEntityResolver > 
& Resolver)
-{
-    maData.mxEntityResolver = Resolver;
-}
-
-void FastSaxParserImpl::setLocale( const lang::Locale & Locale )
-{
-    maData.maLocale = Locale;
-}
-
 void FastSaxParserImpl::setNamespaceHandler( const Reference< 
XFastNamespaceHandler >& Handler )
 {
     maData.mxNamespaceHandler = Handler;
@@ -1400,14 +1384,14 @@ void FastSaxParser::setErrorHandler( const 
uno::Reference< xml::sax::XErrorHandl
     mpImpl->setErrorHandler(Handler);
 }
 
-void FastSaxParser::setEntityResolver( const uno::Reference< 
xml::sax::XEntityResolver >& Resolver )
+void FastSaxParser::setEntityResolver( const uno::Reference< 
xml::sax::XEntityResolver >& )
 {
-    mpImpl->setEntityResolver(Resolver);
+    // not implemented
 }
 
-void FastSaxParser::setLocale( const lang::Locale& rLocale )
+void FastSaxParser::setLocale( const lang::Locale& )
 {
-    mpImpl->setLocale(rLocale);
+    // not implemented
 }
 
 void FastSaxParser::setNamespaceHandler( const uno::Reference< 
css::xml::sax::XFastNamespaceHandler >& Handler)
commit d3d8618608ec0f329a40e029de951374871a8de0
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Fri Feb 22 15:24:59 2019 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sat Feb 23 07:30:03 2019 +0100

    loplugin:unusedfields in sc
    
    Change-Id: I5e9842970821d0ae61e06e3ad35c9f571b43632c
    Reviewed-on: https://gerrit.libreoffice.org/68224
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/sc/inc/externalrefmgr.hxx b/sc/inc/externalrefmgr.hxx
index 0dee986d81c0..00a5ca7429ad 100644
--- a/sc/inc/externalrefmgr.hxx
+++ b/sc/inc/externalrefmgr.hxx
@@ -54,7 +54,7 @@ class ColumnSpanSet;
 class ScExternalRefLink : public ::sfx2::SvBaseLink
 {
 public:
-    ScExternalRefLink(ScDocument* pDoc, sal_uInt16 nFileId, const OUString& 
rFilter);
+    ScExternalRefLink(ScDocument* pDoc, sal_uInt16 nFileId);
     virtual ~ScExternalRefLink() override;
 
     virtual void Closed() override;
@@ -69,7 +69,6 @@ private:
     ScExternalRefLink(const ScExternalRefLink&) = delete;
 
     sal_uInt16 const  mnFileId;
-    OUString    maFilterName;
     ScDocument* const mpDoc;
     bool        mbDoRefresh;
 };
diff --git a/sc/inc/mtvelements.hxx b/sc/inc/mtvelements.hxx
index 04c34775cedb..407f0392c4cd 100644
--- a/sc/inc/mtvelements.hxx
+++ b/sc/inc/mtvelements.hxx
@@ -130,7 +130,6 @@ struct ColumnBlockPosition
 struct ColumnBlockConstPosition
 {
     CellNoteStoreType::const_iterator miCellNotePos;
-    BroadcasterStoreType::const_iterator miBroadcasterPos;
     CellTextAttrStoreType::const_iterator miCellTextAttrPos;
     CellStoreType::const_iterator miCellPos;
 
diff --git a/sc/inc/pagepar.hxx b/sc/inc/pagepar.hxx
index 50a51cde8b63..2a86e32936f2 100644
--- a/sc/inc/pagepar.hxx
+++ b/sc/inc/pagepar.hxx
@@ -59,8 +59,6 @@ struct ScPageAreaParam
     bool    bRepeatRow;
     bool    bRepeatCol;
     ScRange aPrintArea;
-    ScRange aRepeatRow;
-    ScRange aRepeatCol;
 
     ScPageAreaParam();
     ~ScPageAreaParam();
diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx
index a5a6a4a0993a..329971d2b3bf 100644
--- a/sc/source/core/data/column3.cxx
+++ b/sc/source/core/data/column3.cxx
@@ -788,7 +788,6 @@ void ScColumn::InitBlockPosition( sc::ColumnBlockPosition& 
rBlockPos )
 
 void ScColumn::InitBlockPosition( sc::ColumnBlockConstPosition& rBlockPos ) 
const
 {
-    rBlockPos.miBroadcasterPos = maBroadcasters.begin();
     rBlockPos.miCellNotePos = maCellNotes.begin();
     rBlockPos.miCellTextAttrPos = maCellTextAttrs.begin();
     rBlockPos.miCellPos = maCells.begin();
diff --git a/sc/source/core/data/pagepar.cxx b/sc/source/core/data/pagepar.cxx
index 9853e4ac52a5..35a270e39391 100644
--- a/sc/source/core/data/pagepar.cxx
+++ b/sc/source/core/data/pagepar.cxx
@@ -58,8 +58,6 @@ void ScPageAreaParam::Reset()
     bPrintArea = bRepeatRow = bRepeatCol = false;
 
     aPrintArea = ScRange();
-    aRepeatRow = ScRange();
-    aRepeatCol = ScRange();
 }
 
 
diff --git a/sc/source/filter/xml/xmldpimp.hxx 
b/sc/source/filter/xml/xmldpimp.hxx
index c75abd762f41..31bd37690efb 100644
--- a/sc/source/filter/xml/xmldpimp.hxx
+++ b/sc/source/filter/xml/xmldpimp.hxx
@@ -87,8 +87,6 @@ class ScXMLDataPilotTableContext : public ScXMLImportContext
     OUString   sSourceRangeName;
     ScRange         aSourceCellRangeAddress;
     ScRange         aTargetRangeAddress;
-    ScRange         aFilterSourceRange;
-    ScAddress       aFilterOutputPosition;
     ScQueryParam    aSourceQueryParam;
     ScMySourceType  nSourceType;
     sal_uInt32      mnRowFieldCount;
@@ -132,8 +130,6 @@ public:
     void SetSourceRangeName(const OUString& sValue) { sSourceRangeName = 
sValue; bSourceCellRange = true; }
     void SetSourceCellRangeAddress(const ScRange& aValue) { 
aSourceCellRangeAddress = aValue; bSourceCellRange = true; }
     void SetSourceQueryParam(const ScQueryParam& aValue) { aSourceQueryParam = 
aValue; }
-    void SetFilterOutputPosition(const ScAddress& aValue) { 
aFilterOutputPosition = aValue; }
-    void SetFilterSourceRange(const ScRange& aValue) { aFilterSourceRange = 
aValue; }
     void AddDimension(ScDPSaveDimension* pDim);
     void AddGroupDim(const ScDPSaveNumGroupDimension& aNumGroupDim);
     void AddGroupDim(const ScDPSaveGroupDimension& aGroupDim);
diff --git a/sc/source/filter/xml/xmlfilti.cxx 
b/sc/source/filter/xml/xmlfilti.cxx
index ea051785c59d..d97bf44957ee 100644
--- a/sc/source/filter/xml/xmlfilti.cxx
+++ b/sc/source/filter/xml/xmlfilti.cxx
@@ -575,12 +575,8 @@ void SAL_CALL ScXMLDPFilterContext::endFastElement( 
sal_Int32 /*nElement*/ )
     aFilterFields.eSearchType = eSearchType;
     aFilterFields.bCaseSens = bIsCaseSensitive;
     aFilterFields.bDuplicate = !bSkipDuplicates;
-    if (bCopyOutputData)
-        pDataPilotTable->SetFilterOutputPosition(aOutputPosition);
 
     pDataPilotTable->SetSourceQueryParam(aFilterFields);
-    if (bConditionSourceRange)
-        pDataPilotTable->SetFilterSourceRange(aConditionSourceRangeAddress);
 }
 
 void ScXMLDPFilterContext::AddFilterField (const ScQueryEntry& aFilterField)
diff --git a/sc/source/filter/xml/xmltabi.cxx b/sc/source/filter/xml/xmltabi.cxx
index 936170a1624a..ce006dc9e399 100644
--- a/sc/source/filter/xml/xmltabi.cxx
+++ b/sc/source/filter/xml/xmltabi.cxx
@@ -187,7 +187,6 @@ ScXMLTableContext::ScXMLTableContext( ScXMLImport& rImport,
     {
         // This is an external ref cache table.
         pExternalRefInfo.reset(new ScXMLExternalTabData);
-        pExternalRefInfo->maFileUrl = aExtUrl;
         ScDocument* pDoc = GetScImport().GetDocument();
         if (pDoc)
         {
diff --git a/sc/source/filter/xml/xmltabi.hxx b/sc/source/filter/xml/xmltabi.hxx
index 5ada4b365f50..2b194d172d16 100644
--- a/sc/source/filter/xml/xmltabi.hxx
+++ b/sc/source/filter/xml/xmltabi.hxx
@@ -30,7 +30,6 @@ namespace sax_fastparser { class FastAttributeList; }
 
 struct ScXMLExternalTabData
 {
-    OUString maFileUrl;
     ScExternalRefCache::TableTypeRef mpCacheTable;
     sal_Int32 mnRow;
     sal_Int32 mnCol;
diff --git a/sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx 
b/sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx
index 878bb48aed10..81b8cf79460e 100644
--- a/sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx
+++ b/sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx
@@ -584,8 +584,6 @@ struct ScShapeRange
     ScShapeChildVec maBackShapes;
     ScShapeChildVec maForeShapes; // inclusive internal shapes
     ScShapeChildVec maControls;
-    tools::Rectangle       maPixelRect;
-    MapMode         maMapMode;
     ScIAccessibleViewForwarder maViewForwarder;
 };
 
@@ -1010,8 +1008,6 @@ void ScShapeChildren::FillShapes(const tools::Rectangle& 
aPixelPaintRect, const
             tools::Rectangle aRect2(Point(0,0), 
mpAccDoc->GetBoundingBoxOnScreen().GetSize());
             aClippedPixelPaintRect = aPixelPaintRect.GetIntersection(aRect2);
         }
-        maShapeRanges[nRangeId].maPixelRect = aClippedPixelPaintRect;
-        maShapeRanges[nRangeId].maMapMode = aMapMode;
         ScIAccessibleViewForwarder aViewForwarder(mpViewShell, mpAccDoc, 
aMapMode);
         maShapeRanges[nRangeId].maViewForwarder = aViewForwarder;
         const size_t nCount(pPage->GetObjCount());
diff --git a/sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx 
b/sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx
index 0cd1d7ac24d5..4370cbb66db2 100644
--- a/sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx
+++ b/sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx
@@ -284,7 +284,6 @@ void 
ScAccessibleSpreadsheet::ConstructScAccessibleSpreadsheet(
     mnTab = nTab;
     mbDelIns = false;
     mbIsFocusSend = false;
-    maVisCells = GetVisCells(GetVisArea(mpViewShell, meSplitPos));
     if (mpViewShell)
     {
         mpViewShell->AddAccessibilityObject(*this);
diff --git a/sc/source/ui/docshell/externalrefmgr.cxx 
b/sc/source/ui/docshell/externalrefmgr.cxx
index 7f36443d802d..e7fc510ef308 100644
--- a/sc/source/ui/docshell/externalrefmgr.cxx
+++ b/sc/source/ui/docshell/externalrefmgr.cxx
@@ -1411,10 +1411,9 @@ ScExternalRefCache::DocItem* 
ScExternalRefCache::getDocItem(sal_uInt16 nFileId)
     return &itrDoc->second;
 }
 
-ScExternalRefLink::ScExternalRefLink(ScDocument* pDoc, sal_uInt16 nFileId, 
const OUString& rFilter) :
+ScExternalRefLink::ScExternalRefLink(ScDocument* pDoc, sal_uInt16 nFileId) :
     ::sfx2::SvBaseLink(::SfxLinkUpdateMode::ONCALL, 
SotClipboardFormatId::SIMPLE_FILE),
     mnFileId(nFileId),
-    maFilterName(rFilter),
     mpDoc(pDoc),
     mbDoRefresh(true)
 {
@@ -1458,7 +1457,6 @@ void ScExternalRefLink::Closed()
         ScDocShell* pDocShell = ScDocShell::GetViewData()->GetDocShell();
         ScDocShellModificator aMod(*pDocShell);
         pMgr->switchSrcFile(mnFileId, aFile, aFilter);
-        maFilterName = aFilter;
         aMod.SetDocumentModified();
     }
 
@@ -2591,7 +2589,7 @@ void ScExternalRefManager::maybeLinkExternalFile( 
sal_uInt16 nFileId, bool bDefe
         SAL_WARN( "sc.ui", "ScExternalRefManager::maybeLinkExternalFile: 
pLinkMgr==NULL");
         return;
     }
-    ScExternalRefLink* pLink = new ScExternalRefLink(mpDoc, nFileId, aFilter);
+    ScExternalRefLink* pLink = new ScExternalRefLink(mpDoc, nFileId);
     OSL_ENSURE(pFileName, "ScExternalRefManager::maybeLinkExternalFile: file 
name pointer is NULL");
     pLinkMgr->InsertFileLink(*pLink, OBJECT_CLIENT_FILE, *pFileName,
             (aFilter.isEmpty() && bDeferFilterDetection ? nullptr : &aFilter));
diff --git a/sc/source/ui/inc/AccessibleSpreadsheet.hxx 
b/sc/source/ui/inc/AccessibleSpreadsheet.hxx
index b425e92faff1..543526bc596f 100644
--- a/sc/source/ui/inc/AccessibleSpreadsheet.hxx
+++ b/sc/source/ui/inc/AccessibleSpreadsheet.hxx
@@ -216,7 +216,6 @@ private:
     std::unique_ptr<ScRangeList>  mpMarkedRanges;
     ScAccessibleDocument* mpAccDoc;
     rtl::Reference<ScAccessibleCell> mpAccCell;
-    tools::Rectangle       maVisCells;
     ScSplitPos      meSplitPos;
     ScAddress       maActiveCell;
     SCTAB           mnTab;
diff --git a/sc/source/ui/inc/preview.hxx b/sc/source/ui/inc/preview.hxx
index ac4e8f5ed6e8..f3595722398c 100644
--- a/sc/source/ui/inc/preview.hxx
+++ b/sc/source/ui/inc/preview.hxx
@@ -49,7 +49,6 @@ private:
     long            nDisplayStart;      // same as above, relative to the 
start of counting
     DateTime        aDateTime;
     long            nTotalPages;
-    Size            aPageSize;          // for GetOptimalZoom
     ScPrintState    aState;
     std::unique_ptr<ScPreviewLocationData> pLocationData;   // stores table 
layout for accessibility API
     std::unique_ptr<FmFormView> pDrawView;
diff --git a/sc/source/ui/miscdlgs/dataproviderdlg.cxx 
b/sc/source/ui/miscdlgs/dataproviderdlg.cxx
index d55babb13174..38c9d6b4cabd 100644
--- a/sc/source/ui/miscdlgs/dataproviderdlg.cxx
+++ b/sc/source/ui/miscdlgs/dataproviderdlg.cxx
@@ -37,10 +37,6 @@ class ScDataProviderBaseControl : public VclContainer,
     VclPtr<Edit> maEditID;
     VclPtr<PushButton> mpApplyBtn;
 
-    OUString maOldProvider;
-    OUString maURL;
-    OUString maID;
-
     Link<Window*, void> const maImportCallback;
 
     DECL_LINK(ProviderSelectHdl, ListBox&, void);
@@ -163,19 +159,16 @@ void ScDataProviderBaseControl::updateApplyBtn(bool 
bValidConfig)
 IMPL_LINK_NOARG(ScDataProviderBaseControl, ProviderSelectHdl, ListBox&, void)
 {
     isValid();
-    maOldProvider = maProviderList->GetSelectedEntry();
 }
 
 IMPL_LINK_NOARG(ScDataProviderBaseControl, IDEditHdl, Edit&, void)
 {
     isValid();
-    maID = maEditID->GetText();
 }
 
 IMPL_LINK_NOARG(ScDataProviderBaseControl, URLEditHdl, Edit&, void)
 {
     isValid();
-    maURL = maEditURL->GetText();
 }
 
 IMPL_LINK_NOARG(ScDataProviderBaseControl, ApplyBtnHdl, Button*, void)
diff --git a/sc/source/ui/vba/vbaformatcondition.cxx 
b/sc/source/ui/vba/vbaformatcondition.cxx
index ed885a16fa01..51390e200369 100644
--- a/sc/source/ui/vba/vbaformatcondition.cxx
+++ b/sc/source/ui/vba/vbaformatcondition.cxx
@@ -47,7 +47,6 @@ ScVbaFormatCondition::ScVbaFormatCondition( const 
uno::Reference< XHelperInterfa
 {
         mxSheetConditionalEntries = lcl_getScVbaFormatConditionsPtr( 
moFormatConditions )->getSheetConditionalEntries();
 
-        mxSheetConditionalEntry = _xSheetConditionalEntry;
         msStyleName = mxStyle->getName();
 }
 
diff --git a/sc/source/ui/vba/vbaformatcondition.hxx 
b/sc/source/ui/vba/vbaformatcondition.hxx
index 892f6eb6e74e..dbf503bae84c 100644
--- a/sc/source/ui/vba/vbaformatcondition.hxx
+++ b/sc/source/ui/vba/vbaformatcondition.hxx
@@ -31,7 +31,6 @@ typedef ScVbaCondition< ov::excel::XFormatCondition >  
ScVbaFormatCondition_BASE
 class ScVbaFormatCondition final : public ScVbaFormatCondition_BASE
 {
     OUString msStyleName;
-    css::uno::Reference< css::sheet::XSheetConditionalEntry > 
mxSheetConditionalEntry;
     css::uno::Reference< css::sheet::XSheetConditionalEntries > 
mxSheetConditionalEntries;
     css::uno::Reference< ov::excel::XFormatConditions> moFormatConditions;
     css::uno::Reference< ov::excel::XStyle > mxStyle;
diff --git a/sc/source/ui/view/preview.cxx b/sc/source/ui/view/preview.cxx
index 26b1f4fa6825..fe01ce5ad169 100644
--- a/sc/source/ui/view/preview.cxx
+++ b/sc/source/ui/view/preview.cxx
@@ -278,7 +278,6 @@ void ScPreview::CalcPages()
             nTabStart = nThisStart;
 
             aPrintFunc.GetPrintState( aState );
-            aPageSize = aPrintFunc.GetPageSize();
         }
     }
 
diff --git a/sc/source/ui/view/printfun.cxx b/sc/source/ui/view/printfun.cxx
index e144046b7fd7..96248b21bd31 100644
--- a/sc/source/ui/view/printfun.cxx
+++ b/sc/source/ui/view/printfun.cxx
@@ -1032,7 +1032,6 @@ void ScPrintFunc::InitParam( const ScPrintOptions* 
pOptions )
     if ( pRepeatCol )
     {
         aAreaParam.bRepeatCol = true;
-        aAreaParam.aRepeatCol = *pRepeatCol;
         nRepeatStartCol = pRepeatCol->aStart.Col();
         nRepeatEndCol   = pRepeatCol->aEnd  .Col();
     }
@@ -1045,7 +1044,6 @@ void ScPrintFunc::InitParam( const ScPrintOptions* 
pOptions )
     if ( pRepeatRow )
     {
         aAreaParam.bRepeatRow = true;
-        aAreaParam.aRepeatRow = *pRepeatRow;
         nRepeatStartRow = pRepeatRow->aStart.Row();
         nRepeatEndRow   = pRepeatRow->aEnd  .Row();
     }
diff --git a/scripting/source/vbaevents/eventhelper.cxx 
b/scripting/source/vbaevents/eventhelper.cxx
index 405ad6b4747b..b44b9c87acb6 100644
--- a/scripting/source/vbaevents/eventhelper.cxx
+++ b/scripting/source/vbaevents/eventhelper.cxx
@@ -612,7 +612,6 @@ private:
     Reference< frame::XModel > m_xModel;
     bool m_bDocClosed;
     SfxObjectShell* mpShell;
-    OUString msProject;
 };
 
 EventListener::EventListener() :
@@ -620,7 +619,6 @@ OPropertyContainer(GetBroadcastHelper()), 
m_bDocClosed(false), mpShell( nullptr
 {
     registerProperty( EVENTLSTNR_PROPERTY_MODEL, EVENTLSTNR_PROPERTY_ID_MODEL,
         beans::PropertyAttribute::TRANSIENT, &m_xModel, 
cppu::UnoType<decltype(m_xModel)>::get() );
-    msProject = "Standard";
 }
 
 void
@@ -638,14 +636,6 @@ EventListener::setShellFromModel()
         }
         pShell = SfxObjectShell::GetNext( *pShell );
     }
-    // set ProjectName from model
-    try
-    {
-        uno::Reference< beans::XPropertySet > xProps( m_xModel, 
UNO_QUERY_THROW );
-        uno::Reference< script::vba::XVBACompatibility > xVBAMode( 
xProps->getPropertyValue("BasicLibraries"), uno::UNO_QUERY_THROW );
-        msProject = xVBAMode->getProjectName();
-    }
-    catch ( uno::Exception& ) {}
 }
 
 //XEventListener
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to