[Libreoffice-commits] core.git: 2 commits - include/linguistic l10ntools/inc l10ntools/source linguistic/source reportdesign/source sax/source sc/inc scripting/source sc/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Sat Feb 23 06:30:27 UTC 2019
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.grandin at collabora.co.uk>
AuthorDate: Fri Feb 22 15:31:15 2019 +0200
Commit: Noel Grandin <noel.grandin at 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.grandin at 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.grandin at collabora.co.uk>
AuthorDate: Fri Feb 22 15:24:59 2019 +0200
Commit: Noel Grandin <noel.grandin at 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.grandin at 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
More information about the Libreoffice-commits
mailing list