loplugin:unusedfields in sc

Change-Id: If5123e676a27302f3e11475309bc748977c6f430

54 files changed, 75 insertions, 193 deletions

diff --git a/compilerplugins/clang/unusedfields.py b/compilerplugins/clang/unusedfields.py
index 40b5defb999f..4b9b6aa93131 100755
--- a/compilerplugins/clang/unusedfields.py
+++ b/compilerplugins/clang/unusedfields.py
@@ -53,12 +53,16 @@ for d in definitionSet:
         continue
     if d in callSet:
         continue
-    if (definitionToSourceLocationMap[d].startswith("include/")):
+    srcLoc = definitionToSourceLocationMap[d];
+    if (srcLoc.startswith("include/")):
         continue
-    if (definitionToSourceLocationMap[d].startswith("external/")):
+    if (srcLoc.startswith("external/")):
+        continue
+    # this is all representations of on-disk data structures
+    if (srcLoc == "sc/source/filter/inc/scflt.hxx"):
         continue
 
-    tmp1set.add((clazz, definitionToSourceLocationMap[d]))
+    tmp1set.add((clazz, srcLoc))
 
 # sort the results using a "natural order" so sequences like [item1,item2,item10] sort nicely
 def natural_sort_key(s, _nsre=re.compile('([0-9]+)')):
diff --git a/sc/inc/cellvalues.hxx b/sc/inc/cellvalues.hxx
index 7c132b00fce2..9782357dee24 100644
--- a/sc/inc/cellvalues.hxx
+++ b/sc/inc/cellvalues.hxx
@@ -30,9 +30,8 @@ struct CellValueSpan
 {
     SCROW mnRow1;
     SCROW mnRow2;
-    CellType meType;
 
-    CellValueSpan( SCROW nRow1, SCROW nRow2, CellType eType );
+    CellValueSpan( SCROW nRow1, SCROW nRow2 );
 };
 
 /**
diff --git a/sc/inc/compiler.hxx b/sc/inc/compiler.hxx
index 3c8a12e84375..06736b4d2b17 100644
--- a/sc/inc/compiler.hxx
+++ b/sc/inc/compiler.hxx
@@ -96,20 +96,6 @@ protected:
     formula::StackVar eType;
 };
 
-struct ScDoubleRawToken: private ScRawTokenBase
-{
-public:
-    union
-    {   // union only to assure alignment identical to ScRawToken
-        double      nValue;
-        struct {
-            sal_uInt8        cByte;
-            bool        bIsInForceArray;
-        } sbyte;
-    };
-                DECL_FIXEDMEMPOOL_NEWDEL( ScDoubleRawToken );
-};
-
 struct ScRawToken: private ScRawTokenBase
 {
     friend class ScCompiler;
diff --git a/sc/inc/dociter.hxx b/sc/inc/dociter.hxx
index adc76796b4b5..34aa83470003 100644
--- a/sc/inc/dociter.hxx
+++ b/sc/inc/dociter.hxx
@@ -115,20 +115,18 @@ private:
     class DataAccess
     {
     public:
-        DataAccess(const ScDBQueryDataIterator* pParent);
+        DataAccess();
         virtual ~DataAccess() = 0;
         virtual bool getCurrent(Value& rValue) = 0;
         virtual bool getFirst(Value& rValue) = 0;
         virtual bool getNext(Value& rValue) = 0;
-    protected:
-        const ScDBQueryDataIterator* mpParent;
     };
 
     class DataAccessInternal : public DataAccess
     {
         typedef std::pair<sc::CellStoreType::const_iterator,size_t> PositionType;
     public:
-        DataAccessInternal(const ScDBQueryDataIterator* pParent, ScDBQueryParamInternal* pParam, ScDocument* pDoc);
+        DataAccessInternal(ScDBQueryParamInternal* pParam, ScDocument* pDoc);
         virtual ~DataAccessInternal();
         virtual bool getCurrent(Value& rValue) override;
         virtual bool getFirst(Value& rValue) override;
@@ -156,7 +154,7 @@ private:
     class DataAccessMatrix : public DataAccess
     {
     public:
-        DataAccessMatrix(const ScDBQueryDataIterator* pParent, ScDBQueryParamMatrix* pParam);
+        DataAccessMatrix(ScDBQueryParamMatrix* pParam);
         virtual ~DataAccessMatrix();
         virtual bool getCurrent(Value& rValue) override;
         virtual bool getFirst(Value& rValue) override;
diff --git a/sc/inc/document.hxx b/sc/inc/document.hxx
index 72a2d2dcf0cd..781db63bb50a 100644
--- a/sc/inc/document.hxx
+++ b/sc/inc/document.hxx
@@ -394,7 +394,6 @@ private:
     sal_uInt16              nMacroInterpretLevel;           // >0 if macro in interpreter
     sal_uInt16              nInterpreterTableOpLevel;       // >0 if in interpreter TableOp
     sal_uInt16              nSrcVer;                        // file version (load/save)
-    SCROW                   nSrcMaxRow;                     // number of lines to load/save
     sal_uInt16              nFormulaTrackCount;
     HardRecalcState         eHardRecalcState;               // off, temporary, eternal
     SCTAB                   nVisibleTab;                    // for OLE etc., don't use inside ScDocument
@@ -437,11 +436,8 @@ private:
     // for detective update, is set for each change of a formula
     bool                bDetectiveDirty;
 
-    sal_uInt8               nMacroCallMode;     // Macros per warning dialog disabled?
     bool                bHasMacroFunc;      // valid only after loading
 
-    sal_uInt8               nVisSpellState;
-
     sal_uInt8               nAsianCompression;
     sal_uInt8               nAsianKerning;
 
diff --git a/sc/inc/documentlinkmgr.hxx b/sc/inc/documentlinkmgr.hxx
index f5ba279423a1..3f9cc0c7a4cd 100644
--- a/sc/inc/documentlinkmgr.hxx
+++ b/sc/inc/documentlinkmgr.hxx
@@ -34,7 +34,7 @@ class DocumentLinkManager : boost::noncopyable
     std::unique_ptr<DocumentLinkManagerImpl> mpImpl;
 
 public:
-    DocumentLinkManager( ScDocument& rDoc, SfxObjectShell* pShell );
+    DocumentLinkManager( SfxObjectShell* pShell );
     ~DocumentLinkManager();
 
     void setDataStream( DataStream* p );
diff --git a/sc/inc/dpresfilter.hxx b/sc/inc/dpresfilter.hxx
index 0bcdc1a43953..9ff9ed4eee0d 100644
--- a/sc/inc/dpresfilter.hxx
+++ b/sc/inc/dpresfilter.hxx
@@ -60,10 +60,9 @@ private:
 
     struct DimensionNode : boost::noncopyable
     {
-        const MemberNode* mpParent;
         MembersType maChildMembers;
 
-        DimensionNode(const MemberNode* pParent);
+        DimensionNode();
         ~DimensionNode();
 
 #if DEBUG_PIVOT_TABLE
@@ -73,11 +72,10 @@ private:
 
     struct MemberNode : boost::noncopyable
     {
-        const DimensionNode* mpParent;
         ValuesType maValues;
         DimensionsType maChildDimensions;
 
-        MemberNode(const DimensionNode* pParent);
+        MemberNode();
         ~MemberNode();
 
 #if DEBUG_PIVOT_TABLE
diff --git a/sc/inc/dptabdat.hxx b/sc/inc/dptabdat.hxx
index d47951e47d10..6a013f55a122 100644
--- a/sc/inc/dptabdat.hxx
+++ b/sc/inc/dptabdat.hxx
@@ -93,8 +93,6 @@ public:
         ScDPResultMember*               pColRoot;
         ScDPResultMember*               pRowRoot;
 
-        bool                            bRepeatIfEmpty;
-
         CalcInfo();
     };
 
diff --git a/sc/inc/externalrefmgr.hxx b/sc/inc/externalrefmgr.hxx
index ae4decdda6c2..cb8a1b296a2d 100644
--- a/sc/inc/externalrefmgr.hxx
+++ b/sc/inc/externalrefmgr.hxx
@@ -430,7 +430,6 @@ public:
         OUString maRelativeName;
         OUString maFilterName;
         OUString maFilterOptions;
-        bool bUnsaved;
 
         void maybeCreateRealFileName(const OUString& rOwnDocName);
     };
diff --git a/sc/inc/formulacell.hxx b/sc/inc/formulacell.hxx
index 5a6f87b1eb25..5424cb48252f 100644
--- a/sc/inc/formulacell.hxx
+++ b/sc/inc/formulacell.hxx
@@ -72,7 +72,6 @@ public:
     bool mbSubTotal:1;
 
     sal_uInt8 meCalcState;
-    sal_uInt8 meKernelState;
 
     ScFormulaCellGroup();
     ~ScFormulaCellGroup();
diff --git a/sc/source/core/data/cellvalues.cxx b/sc/source/core/data/cellvalues.cxx
index bdee2b95afac..049538da5f99 100644
--- a/sc/source/core/data/cellvalues.cxx
+++ b/sc/source/core/data/cellvalues.cxx
@@ -24,29 +24,10 @@ struct BlockPos
     size_t mnEnd;
 };
 
-CellType toCellType( mdds::mtv::element_t eType )
-{
-    switch (eType)
-    {
-        case element_type_numeric:
-            return CELLTYPE_VALUE;
-        case element_type_string:
-            return CELLTYPE_STRING;
-        case element_type_edittext:
-            return CELLTYPE_EDIT;
-        case element_type_formula:
-            return CELLTYPE_FORMULA;
-        default:
-            ;
-    }
-
-    return CELLTYPE_NONE;
-}
-
 }
 
-CellValueSpan::CellValueSpan( SCROW nRow1, SCROW nRow2, CellType eType ) :
-    mnRow1(nRow1), mnRow2(nRow2), meType(eType) {}
+CellValueSpan::CellValueSpan( SCROW nRow1, SCROW nRow2 ) :
+    mnRow1(nRow1), mnRow2(nRow2) {}
 
 struct CellValuesImpl : boost::noncopyable
 {
@@ -163,7 +144,7 @@ std::vector<CellValueSpan> CellValues::getNonEmptySpans() const
             // Record this span.
             size_t nRow1 = it->position;
             size_t nRow2 = nRow1 + it->size - 1;
-            aRet.push_back(CellValueSpan(nRow1, nRow2, toCellType(it->type)));
+            aRet.push_back(CellValueSpan(nRow1, nRow2));
         }
     }
     return aRet;
diff --git a/sc/source/core/data/dociter.cxx b/sc/source/core/data/dociter.cxx
index 8e3234d258d4..77ac93a5f198 100644
--- a/sc/source/core/data/dociter.cxx
+++ b/sc/source/core/data/dociter.cxx
@@ -308,8 +308,7 @@ bool ScValueIterator::GetNext(double& rValue, sal_uInt16& rErr)
     return GetThis(rValue, rErr);
 }
 
-ScDBQueryDataIterator::DataAccess::DataAccess(const ScDBQueryDataIterator* pParent) :
-    mpParent(pParent)
+ScDBQueryDataIterator::DataAccess::DataAccess()
 {
 }
 
@@ -342,8 +341,8 @@ bool ScDBQueryDataIterator::IsQueryValid(
     return rDoc.maTabs[nTab]->ValidQuery(nRow, rParam, pCell);
 }
 
-ScDBQueryDataIterator::DataAccessInternal::DataAccessInternal(const ScDBQueryDataIterator* pParent, ScDBQueryParamInternal* pParam, ScDocument* pDoc)
-    : DataAccess(pParent)
+ScDBQueryDataIterator::DataAccessInternal::DataAccessInternal(ScDBQueryParamInternal* pParam, ScDocument* pDoc)
+    : DataAccess()
     , mpCells(nullptr)
     , mpParam(pParam)
     , mpDoc(pDoc)
@@ -522,8 +521,8 @@ void ScDBQueryDataIterator::DataAccessInternal::incPos()
         incBlock();
 }
 
-ScDBQueryDataIterator::DataAccessMatrix::DataAccessMatrix(const ScDBQueryDataIterator* pParent, ScDBQueryParamMatrix* pParam)
-    : DataAccess(pParent)
+ScDBQueryDataIterator::DataAccessMatrix::DataAccessMatrix(ScDBQueryParamMatrix* pParam)
+    : DataAccess()
     , mpParam(pParam)
     , mnCurRow(0)
 {
@@ -764,13 +763,13 @@ ScDBQueryDataIterator::ScDBQueryDataIterator(ScDocument* pDocument, ScDBQueryPar
         case ScDBQueryParamBase::INTERNAL:
         {
             ScDBQueryParamInternal* p = static_cast<ScDBQueryParamInternal*>(pParam);
-            mpData.reset(new DataAccessInternal(this, p, pDocument));
+            mpData.reset(new DataAccessInternal(p, pDocument));
         }
         break;
         case ScDBQueryParamBase::MATRIX:
         {
             ScDBQueryParamMatrix* p = static_cast<ScDBQueryParamMatrix*>(pParam);
-            mpData.reset(new DataAccessMatrix(this, p));
+            mpData.reset(new DataAccessMatrix(p));
         }
     }
 }
diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx
index dd554ec7e462..d27a5e6d63c3 100644
--- a/sc/source/core/data/documen2.cxx
+++ b/sc/source/core/data/documen2.cxx
@@ -176,7 +176,6 @@ ScDocument::ScDocument( ScDocumentMode eMode, SfxObjectShell* pDocShell ) :
         nMacroInterpretLevel(0),
         nInterpreterTableOpLevel(0),
         nSrcVer( SC_CURRENT_VERSION ),
-        nSrcMaxRow( MAXROW ),
         nFormulaTrackCount(0),
         eHardRecalcState(HARDRECALCSTATE_OFF),
         nVisibleTab( 0 ),
@@ -201,9 +200,7 @@ ScDocument::ScDocument( ScDocumentMode eMode, SfxObjectShell* pDocShell ) :
         bInDtorClear( false ),
         bExpandRefs( false ),
         bDetectiveDirty( false ),
-        nMacroCallMode( SC_MACROCALL_ALLOWED ),
         bHasMacroFunc( false ),
-        nVisSpellState( 0 ),
         nAsianCompression(SC_ASIANCOMPRESSION_INVALID),
         nAsianKerning(SC_ASIANKERNING_INVALID),
         bPastingDrawFromOtherDoc( false ),
@@ -266,7 +263,7 @@ const sfx2::LinkManager* ScDocument::GetLinkManager() const
 sc::DocumentLinkManager& ScDocument::GetDocLinkManager()
 {
     if (!mpDocLinkMgr)
-        mpDocLinkMgr.reset(new sc::DocumentLinkManager(*this, pShell));
+        mpDocLinkMgr.reset(new sc::DocumentLinkManager(pShell));
     return *mpDocLinkMgr;
 }
 
diff --git a/sc/source/core/data/dpresfilter.cxx b/sc/source/core/data/dpresfilter.cxx
index 603ab14b5a84..a416bc3a527e 100644
--- a/sc/source/core/data/dpresfilter.cxx
+++ b/sc/source/core/data/dpresfilter.cxx
@@ -30,8 +30,7 @@ size_t ScDPResultTree::NamePairHash::operator() (const NamePairType& rPair) cons
     return aHash(rPair.first) + aHash(rPair.second);
 }
 
-ScDPResultTree::DimensionNode::DimensionNode(const MemberNode* pParent) :
-    mpParent(pParent) {}
+ScDPResultTree::DimensionNode::DimensionNode() {}
 
 ScDPResultTree::DimensionNode::~DimensionNode()
 {
@@ -60,8 +59,7 @@ void ScDPResultTree::DimensionNode::dump(int nLevel) const
 }
 #endif
 
-ScDPResultTree::MemberNode::MemberNode(const DimensionNode* pParent) :
-    mpParent(pParent) {}
+ScDPResultTree::MemberNode::MemberNode() {}
 
 ScDPResultTree::MemberNode::~MemberNode()
 {
@@ -87,7 +85,7 @@ void ScDPResultTree::MemberNode::dump(int nLevel) const
 }
 #endif
 
-ScDPResultTree::ScDPResultTree() : mpRoot(new MemberNode(nullptr)) {}
+ScDPResultTree::ScDPResultTree() : mpRoot(new MemberNode) {}
 ScDPResultTree::~ScDPResultTree()
 {
     delete mpRoot;
@@ -120,7 +118,7 @@ void ScDPResultTree::add(
         {
             // New dimenison.  Insert it.
             std::pair<DimensionsType::iterator, bool> r =
-                rDims.insert(DimensionsType::value_type(aUpperName, new DimensionNode(pMemNode)));
+                rDims.insert(DimensionsType::value_type(aUpperName, new DimensionNode));
 
             if (!r.second)
                 // Insertion failed!
@@ -141,7 +139,7 @@ void ScDPResultTree::add(
             // New member.  Insert it.
             std::pair<MembersType::iterator, bool> r =
                 rMembers.insert(
-                    MembersType::value_type(aUpperName, new MemberNode(pDim)));
+                    MembersType::value_type(aUpperName, new MemberNode));
 
             if (!r.second)
                 // Insertion failed!
@@ -192,7 +190,7 @@ void ScDPResultTree::clear()
 {
     maPrimaryDimName = EMPTY_OUSTRING;
     delete mpRoot;
-    mpRoot = new MemberNode(nullptr);
+    mpRoot = new MemberNode;
 }
 
 const ScDPResultTree::ValuesType* ScDPResultTree::getResults(
diff --git a/sc/source/core/data/dptabdat.cxx b/sc/source/core/data/dptabdat.cxx
index bee083a1c800..9d9fac389cfb 100644
--- a/sc/source/core/data/dptabdat.cxx
+++ b/sc/source/core/data/dptabdat.cxx
@@ -41,8 +41,7 @@ using ::std::vector;
 ScDPTableData::CalcInfo::CalcInfo() :
     pInitState( nullptr ),
     pColRoot( nullptr ),
-    pRowRoot( nullptr ),
-    bRepeatIfEmpty(false)
+    pRowRoot( nullptr )
 {
 }
 
diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx
index a405ebe6f8dd..94250a1ce2f8 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -548,8 +548,7 @@ ScFormulaCellGroup::ScFormulaCellGroup() :
     mnFormatType(css::util::NumberFormat::NUMBER),
     mbInvariant(false),
     mbSubTotal(false),
-    meCalcState(sc::GroupCalcEnabled),
-    meKernelState(sc::OpenCLKernelNone)
+    meCalcState(sc::GroupCalcEnabled)
 {
     SAL_INFO( "sc.core.formulacell", "ScFormulaCellGroup ctor this " << this);
 }
diff --git a/sc/source/core/inc/doubleref.hxx b/sc/source/core/inc/doubleref.hxx
index 9fbb62d067b1..bf601e2ac28d 100644
--- a/sc/source/core/inc/doubleref.hxx
+++ b/sc/source/core/inc/doubleref.hxx
@@ -74,7 +74,7 @@ public:
     virtual bool isRangeEqual(const ScRange& rRange) const = 0;
 
 protected:
-    ScDBRangeBase(ScDocument* pDoc, RefType eType);
+    ScDBRangeBase(ScDocument* pDoc);
     ScDocument* getDoc() const { return mpDoc;}
 
     /**
@@ -87,7 +87,6 @@ private:
     ScDBRangeBase(); // disabled
 
     ScDocument* mpDoc;
-    RefType meType;
 };
 
 class ScDBInternalRange : public ScDBRangeBase
diff --git a/sc/source/core/tool/doubleref.cxx b/sc/source/core/tool/doubleref.cxx
index f60e8da195a2..5089caaec7e1 100644
--- a/sc/source/core/tool/doubleref.cxx
+++ b/sc/source/core/tool/doubleref.cxx
@@ -241,8 +241,8 @@ bool lcl_fillQueryEntries(
 
 }
 
-ScDBRangeBase::ScDBRangeBase(ScDocument* pDoc, RefType eType) :
-    mpDoc(pDoc), meType(eType)
+ScDBRangeBase::ScDBRangeBase(ScDocument* pDoc) :
+    mpDoc(pDoc)
 {
 }
 
@@ -269,7 +269,7 @@ void ScDBRangeBase::fillQueryOptions(ScQueryParamBase* pParam)
 }
 
 ScDBInternalRange::ScDBInternalRange(ScDocument* pDoc, const ScRange& rRange) :
-    ScDBRangeBase(pDoc, INTERNAL), maRange(rRange)
+    ScDBRangeBase(pDoc), maRange(rRange)
 {
 }
 
@@ -384,7 +384,7 @@ bool ScDBInternalRange::isRangeEqual(const ScRange& rRange) const
 }
 
 ScDBExternalRange::ScDBExternalRange(ScDocument* pDoc, const ScMatrixRef& pMat) :
-    ScDBRangeBase(pDoc, EXTERNAL), mpMatrix(pMat)
+    ScDBRangeBase(pDoc), mpMatrix(pMat)
 {
     SCSIZE nC, nR;
     mpMatrix->GetDimensions(nC, nR);
diff --git a/sc/source/core/tool/token.cxx b/sc/source/core/tool/token.cxx
index 7c43d37be37e..498ba94c0c5d 100644
--- a/sc/source/core/tool/token.cxx
+++ b/sc/source/core/tool/token.cxx
@@ -197,8 +197,6 @@ namespace
 // Since RawTokens are temporary for the compiler, don't align on 4k and waste memory.
 // ScRawToken size is FixMembers + MAXSTRLEN + ~4 ~= 1036
 IMPL_FIXEDMEMPOOL_NEWDEL( ScRawToken )
-// Some ScDoubleRawToken, FixMembers + sizeof(double) ~= 16
-IMPL_FIXEDMEMPOOL_NEWDEL( ScDoubleRawToken )
 
 // Need a whole bunch of ScSingleRefToken
 IMPL_FIXEDMEMPOOL_NEWDEL( ScSingleRefToken )
diff --git a/sc/source/filter/excel/frmbase.cxx b/sc/source/filter/excel/frmbase.cxx
index 2284502dbe87..4e77af817f27 100644
--- a/sc/source/filter/excel/frmbase.cxx
+++ b/sc/source/filter/excel/frmbase.cxx
@@ -168,8 +168,7 @@ const ScRange* _ScRangeListTabs::Next ()
 ConverterBase::ConverterBase( svl::SharedStringPool& rSPool, sal_uInt16 nNewBuffer ) :
     aPool(rSPool),
     aEingPos( 0, 0, 0 ),
-    eStatus( ConvOK ),
-    nBufferSize( nNewBuffer )
+    eStatus( ConvOK )
 {
     OSL_ENSURE( nNewBuffer > 0, "ConverterBase::ConverterBase - nNewBuffer == 0!" );
     pBuffer = new sal_Char[ nNewBuffer ];
diff --git a/sc/source/filter/html/htmlexp.cxx b/sc/source/filter/html/htmlexp.cxx
index 3e2e8c81e318..12ce2a120229 100644
--- a/sc/source/filter/html/htmlexp.cxx
+++ b/sc/source/filter/html/htmlexp.cxx
@@ -208,7 +208,6 @@ ScHTMLExport::ScHTMLExport( SvStream& rStrmP, const OUString& rBaseURL, ScDocume
     ScExportBase( rStrmP, pDocP, rRangeP ),
     aBaseURL( rBaseURL ),
     aStreamPath( rStreamPathP ),
-    aFilterOptions( rFilterOptions ),
     pAppWin( Application::GetDefaultDevice() ),
     nUsedTables( 0 ),
     nIndent( 0 ),
diff --git a/sc/source/filter/html/htmlpars.cxx b/sc/source/filter/html/htmlpars.cxx
index a59c4e3a9444..8e906bd43f94 100644
--- a/sc/source/filter/html/htmlpars.cxx
+++ b/sc/source/filter/html/htmlpars.cxx
@@ -1070,7 +1070,7 @@ void ScHTMLLayoutParser::TableOn( ImportInfo* pInfo )
         Colonize( pActEntry );
         aTableStack.push( new ScHTMLTableStackEntry(
             pActEntry, xLockedList, pLocalColOffset, nFirstTableCell,
-            nColCnt, nRowCnt, nColCntStart, nMaxCol, nTable,
+            nRowCnt, nColCntStart, nMaxCol, nTable,
             nTableWidth, nColOffset, nColOffsetStart,
             bFirstRow ) );
         sal_uInt16 nLastWidth = nTableWidth;
@@ -1132,7 +1132,7 @@ void ScHTMLLayoutParser::TableOn( ImportInfo* pInfo )
         }
         aTableStack.push( new ScHTMLTableStackEntry(
             pActEntry, xLockedList, pLocalColOffset, nFirstTableCell,
-            nColCnt, nRowCnt, nColCntStart, nMaxCol, nTable,
+            nRowCnt, nColCntStart, nMaxCol, nTable,
             nTableWidth, nColOffset, nColOffsetStart,
             bFirstRow ) );
         // As soon as we have multiple tables we need to be tolerant with the offsets.
diff --git a/sc/source/filter/inc/autofilterbuffer.hxx b/sc/source/filter/inc/autofilterbuffer.hxx
index 3eb2b2befcf6..21f3bab6ea71 100644
--- a/sc/source/filter/inc/autofilterbuffer.hxx
+++ b/sc/source/filter/inc/autofilterbuffer.hxx
@@ -115,7 +115,6 @@ struct FilterCriterionModel
     css::uno::Any       maValue; /// Comparison operand.
     sal_Int32           mnOperator;     /// Comparison operator.
     sal_uInt8           mnDataType;     /// Operand data type (BIFF only).
-    sal_uInt8           mnStrLen;       /// Length of string operand (BIFF5-BIFF8 only).
 
     explicit            FilterCriterionModel();
 
diff --git a/sc/source/filter/inc/commentsbuffer.hxx b/sc/source/filter/inc/commentsbuffer.hxx
index af5759b3c4e6..b5c8f3ae0386 100644
--- a/sc/source/filter/inc/commentsbuffer.hxx
+++ b/sc/source/filter/inc/commentsbuffer.hxx
@@ -32,9 +32,7 @@ struct CommentModel
     css::table::CellRangeAddress
                         maRange;            /// Position of the comment in the worksheet.
     RichStringRef       mxText;             /// Formatted text of the comment (not used in BIFF8).
-    OUString            maAuthor;           /// Comment author (BIFF8 only).
     sal_Int32           mnAuthorId;         /// Identifier of the comment's author (OOXML and BIFF12 only).
-    sal_uInt16          mnObjId;            /// Drawing object identifier (BIFF8 only).
     bool                mbAutoFill;         /// Auto Selection of comment object's fill style
     bool                mbAutoScale;        /// Auto Scale comment text
     bool                mbColHidden;        /// Comment cell's Column is Hidden
diff --git a/sc/source/filter/inc/externallinkbuffer.hxx b/sc/source/filter/inc/externallinkbuffer.hxx
index 526af61f312c..d22e91b18a41 100644
--- a/sc/source/filter/inc/externallinkbuffer.hxx
+++ b/sc/source/filter/inc/externallinkbuffer.hxx
@@ -40,7 +40,6 @@ namespace xls {
 
 struct ExternalNameModel
 {
-    bool                mbBuiltIn;          /// Name is a built-in name.
     bool                mbNotify;           /// Notify application on data change.
     bool                mbPreferPic;        /// Picture link.
     bool                mbStdDocName;       /// Name is the StdDocumentName for DDE.
@@ -270,7 +269,6 @@ private:
     OUString            maTargetUrl;        /// Target link, DDE topic, OLE target.
     css::uno::Reference< css::sheet::XExternalDocLink >
                         mxDocLink;          /// Interface for an external document.
-    Int16Vector         maCalcSheets;       /// Internal sheet indexes.
     Int32Vector         maSheetCaches;      /// External sheet cache indexes.
     ExternalNameVector  maExtNames;         /// Defined names in external document.
 };
diff --git a/sc/source/filter/inc/formel.hxx b/sc/source/filter/inc/formel.hxx
index 35a2c8557216..e5d1e463961b 100644
--- a/sc/source/filter/inc/formel.hxx
+++ b/sc/source/filter/inc/formel.hxx
@@ -86,7 +86,6 @@ protected:
     ScAddress           aEingPos;
     ConvErr             eStatus;
     sal_Char*           pBuffer;        // universal buffer
-    sal_uInt16              nBufferSize;    // ...and its size
 
     ConverterBase( svl::SharedStringPool& rSPool, sal_uInt16 nNewBuffer );
     virtual             ~ConverterBase();
diff --git a/sc/source/filter/inc/formulabase.hxx b/sc/source/filter/inc/formulabase.hxx
index 65eb0022935a..174190f0f51c 100644
--- a/sc/source/filter/inc/formulabase.hxx
+++ b/sc/source/filter/inc/formulabase.hxx
@@ -477,8 +477,6 @@ enum FuncParamConversion
 struct FunctionParamInfo
 {
     FuncParamValidity   meValid;        /// Parameter validity.
-    FuncParamConversion meConv;         /// Token class conversion type.
-    bool                mbValType;      /// Data type (false = REFTYPE, true = VALTYPE).
 };
 
 // Function data ==============================================================
diff --git a/sc/source/filter/inc/htmlexp.hxx b/sc/source/filter/inc/htmlexp.hxx
index 48f72035aef8..3e147e0f51d8 100644
--- a/sc/source/filter/inc/htmlexp.hxx
+++ b/sc/source/filter/inc/htmlexp.hxx
@@ -108,7 +108,6 @@ class ScHTMLExport : public ScExportBase
     ScHTMLStyle      aHTMLStyle;
     OUString         aBaseURL;
     OUString         aStreamPath;
-    OUString         aFilterOptions;
     OUString         aCId;           // Content-Id for Mail-Export
     VclPtr<OutputDevice> pAppWin;        // for Pixel-work
     FileNameMapPtr   pFileNameMap;        // for CopyLocalFileToINet
diff --git a/sc/source/filter/inc/htmlpars.hxx b/sc/source/filter/inc/htmlpars.hxx
index eb00077533e4..1e71c8bd89fe 100644
--- a/sc/source/filter/inc/htmlpars.hxx
+++ b/sc/source/filter/inc/htmlpars.hxx
@@ -102,27 +102,26 @@ struct ScHTMLTableStackEntry
     ScRangeListRef      xLockedList;
     ScEEParseEntry*     pCellEntry;
     ScHTMLColOffset*    pLocalColOffset;
-    sal_uLong               nFirstTableCell;
-    SCCOL               nColCnt;
+    sal_uLong           nFirstTableCell;
     SCROW               nRowCnt;
     SCCOL               nColCntStart;
     SCCOL               nMaxCol;
-    sal_uInt16              nTable;
-    sal_uInt16              nTableWidth;
-    sal_uInt16              nColOffset;
-    sal_uInt16              nColOffsetStart;
+    sal_uInt16          nTable;
+    sal_uInt16          nTableWidth;
+    sal_uInt16          nColOffset;
+    sal_uInt16          nColOffsetStart;
     bool                bFirstRow;
                         ScHTMLTableStackEntry( ScEEParseEntry* pE,
                                 const ScRangeListRef& rL, ScHTMLColOffset* pTO,
                                 sal_uLong nFTC,
-                                SCCOL nCol, SCROW nRow,
+                                SCROW nRow,
                                 SCCOL nStart, SCCOL nMax, sal_uInt16 nTab,
                                 sal_uInt16 nTW, sal_uInt16 nCO, sal_uInt16 nCOS,
                                 bool bFR )
                             : xLockedList( rL ), pCellEntry( pE ),
                             pLocalColOffset( pTO ),
                             nFirstTableCell( nFTC ),
-                            nColCnt( nCol ), nRowCnt( nRow ),
+                            nRowCnt( nRow ),
                             nColCntStart( nStart ), nMaxCol( nMax ),
                             nTable( nTab ), nTableWidth( nTW ),
                             nColOffset( nCO ), nColOffsetStart( nCOS ),
diff --git a/sc/source/filter/inc/pivottablebuffer.hxx b/sc/source/filter/inc/pivottablebuffer.hxx
index aaef9bb1bdca..37ee04edf586 100644
--- a/sc/source/filter/inc/pivottablebuffer.hxx
+++ b/sc/source/filter/inc/pivottablebuffer.hxx
@@ -239,8 +239,6 @@ struct PTDefinitionModel : public AutoFormatModel
     sal_Int32           mnPageWrap;
     sal_Int32           mnIndent;
     sal_Int32           mnChartFormat;
-    sal_uInt16          mnRowFields;
-    sal_uInt16          mnColFields;
     bool                mbDataOnRows;
     bool                mbShowError;
     bool                mbShowMissing;
diff --git a/sc/source/filter/inc/worksheetbuffer.hxx b/sc/source/filter/inc/worksheetbuffer.hxx
index 148f44770c48..cc865595fbd4 100644
--- a/sc/source/filter/inc/worksheetbuffer.hxx
+++ b/sc/source/filter/inc/worksheetbuffer.hxx
@@ -38,11 +38,10 @@ struct SheetInfoModel
 {
     OUString     maRelId;        /// Relation identifier for the sheet substream.
     OUString     maName;         /// Original name of the sheet.
-    sal_Int64           mnBiffHandle;   /// BIFF record handle of the sheet substream.
-    sal_Int32           mnSheetId;      /// Sheet identifier.
-    sal_Int32           mnState;        /// Visibility state.
+    sal_Int32    mnSheetId;      /// Sheet identifier.
+    sal_Int32    mnState;        /// Visibility state.
 
-    explicit            SheetInfoModel();
+    explicit     SheetInfoModel();
 };
 
 /** Stores information about all sheets in a spreadsheet document.
diff --git a/sc/source/filter/inc/xltracer.hxx b/sc/source/filter/inc/xltracer.hxx
index 8798597c65e9..6b216df04f27 100644
--- a/sc/source/filter/inc/xltracer.hxx
+++ b/sc/source/filter/inc/xltracer.hxx
@@ -60,15 +60,6 @@ enum XclTracerId
     eTraceLength         /// this *should* always be the final entry
 };
 
-struct XclTracerDetails
-{
-    XclTracerId                 meProblemId;    /// Excel Import Trace index.
-    sal_uInt32                  mnID;           /// actual ID Index trace tag Value
-    const sal_Char*             mpContext;      /// Context for problem e.g. Limits
-    const sal_Char*             mpDetail;       /// Context Detail e.g. SheetX
-    const sal_Char*             mpProblem;      /// Description of problem
-};
-
 /** This class wraps an MSFilterTracer to create trace logs for import/export filters. */
 class XclTracer
 {
diff --git a/sc/source/filter/lotus/lotfilter.hxx b/sc/source/filter/lotus/lotfilter.hxx
index 85acbdc9ac83..7a7f9f9128a0 100644
--- a/sc/source/filter/lotus/lotfilter.hxx
+++ b/sc/source/filter/lotus/lotfilter.hxx
@@ -47,7 +47,6 @@ struct LotusContext
 
     SvxHorJustifyItem *pAttrRight, *pAttrLeft, *pAttrCenter, *pAttrRepeat, *pAttrStandard;
     ScProtectionAttr* pAttrUnprot;
-    SfxUInt32Item**   pAttrValForms;
 
     FormCache*       pValueFormCache; // -> in memory.cxx initialisiert
 
@@ -66,7 +65,6 @@ struct LotusContext
         , pAttrRepeat(nullptr)
         , pAttrStandard(nullptr)
         , pAttrUnprot(nullptr)
-        , pAttrValForms(nullptr)
         , pValueFormCache(nullptr)
     {
     }
diff --git a/sc/source/filter/oox/autofilterbuffer.cxx b/sc/source/filter/oox/autofilterbuffer.cxx
index cd03e08b0c3a..b659394acb0f 100644
--- a/sc/source/filter/oox/autofilterbuffer.cxx
+++ b/sc/source/filter/oox/autofilterbuffer.cxx
@@ -318,8 +318,7 @@ ApiFilterSettings Top10Filter::finalizeImport( sal_Int32 /*nMaxCount*/ )
 
 FilterCriterionModel::FilterCriterionModel() :
     mnOperator( XML_equal ),
-    mnDataType( BIFF_FILTER_DATATYPE_NONE ),
-    mnStrLen( 0 )
+    mnDataType( BIFF_FILTER_DATATYPE_NONE )
 {
 }
 
diff --git a/sc/source/filter/oox/commentsbuffer.cxx b/sc/source/filter/oox/commentsbuffer.cxx
index d126cf02098a..7198dc1ea8ab 100644
--- a/sc/source/filter/oox/commentsbuffer.cxx
+++ b/sc/source/filter/oox/commentsbuffer.cxx
@@ -83,7 +83,6 @@ static sal_Int32 lcl_ToVertAlign( sal_Int32 nAlign )
 
 CommentModel::CommentModel()
     : mnAuthorId(-1)
-    , mnObjId(BIFF_OBJ_INVALID_ID)
     , mbAutoFill(false)
     , mbAutoScale(false)
     , mbColHidden(false)
diff --git a/sc/source/filter/oox/externallinkbuffer.cxx b/sc/source/filter/oox/externallinkbuffer.cxx
index e57cf73844f7..5b4a314f9477 100644
--- a/sc/source/filter/oox/externallinkbuffer.cxx
+++ b/sc/source/filter/oox/externallinkbuffer.cxx
@@ -66,7 +66,6 @@ const sal_uInt16 BIFF12_EXTNAME_ICONIFIED   = 0x0020;
 } // namespace
 
 ExternalNameModel::ExternalNameModel() :
-    mbBuiltIn( false ),
     mbNotify( false ),
     mbPreferPic( false ),
     mbStdDocName( false ),
diff --git a/sc/source/filter/oox/formulabase.cxx b/sc/source/filter/oox/formulabase.cxx
index f216b77c7b95..b17a896f4ba9 100644
--- a/sc/source/filter/oox/formulabase.cxx
+++ b/sc/source/filter/oox/formulabase.cxx
@@ -265,18 +265,18 @@ const sal_uInt8 V = BIFF_TOKCLASS_VAL;
 const sal_uInt8 A = BIFF_TOKCLASS_ARR;
 
 // abbreviations for parameter infos
-#define RO   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_ORG, false }
-#define RA   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_ARR, false }
-#define RR   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_RPT, false }
-#define RX   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_RPX, false }
-#define VO   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_ORG, true  }
-#define VV   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_VAL, true  }
-#define VA   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_ARR, true  }
-#define VR   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_RPT, true  }
-#define VX   { FUNC_PARAM_REGULAR,   FUNC_PARAMCONV_RPX, true  }
-#define RO_E { FUNC_PARAM_EXCELONLY, FUNC_PARAMCONV_ORG, false }
-#define VR_E { FUNC_PARAM_EXCELONLY, FUNC_PARAMCONV_RPT, true  }
-#define C    { FUNC_PARAM_CALCONLY,  FUNC_PARAMCONV_ORG, false }
+#define RO   { FUNC_PARAM_REGULAR }
+#define RA   { FUNC_PARAM_REGULAR }
+#define RR   { FUNC_PARAM_REGULAR }
+#define RX   { FUNC_PARAM_REGULAR }
+#define VO   { FUNC_PARAM_REGULAR  }
+#define VV   { FUNC_PARAM_REGULAR  }
+#define VA   { FUNC_PARAM_REGULAR  }
+#define VR   { FUNC_PARAM_REGULAR  }
+#define VX   { FUNC_PARAM_REGULAR  }
+#define RO_E { FUNC_PARAM_EXCELONLY }
+#define VR_E { FUNC_PARAM_EXCELONLY  }
+#define C    { FUNC_PARAM_CALCONLY }
 
 // Note: parameter types of all macro sheet functions (FUNCFLAG_MACROFUNC/FUNCFLAG_MACROCMD) untested!
 
diff --git a/sc/source/filter/oox/pivottablebuffer.cxx b/sc/source/filter/oox/pivottablebuffer.cxx
index 4bed45cf4342..7756adad72d0 100644
--- a/sc/source/filter/oox/pivottablebuffer.cxx
+++ b/sc/source/filter/oox/pivottablebuffer.cxx
@@ -862,8 +862,6 @@ PTDefinitionModel::PTDefinitionModel() :
     mnPageWrap( 0 ),
     mnIndent( 1 ),
     mnChartFormat( 0 ),
-    mnRowFields( 0 ),
-    mnColFields( 0 ),
     mbDataOnRows( false ),
     mbShowError( false ),
     mbShowMissing( true ),
diff --git a/sc/source/filter/oox/worksheetbuffer.cxx b/sc/source/filter/oox/worksheetbuffer.cxx
index 6c3a0dd12182..e4319194e785 100644
--- a/sc/source/filter/oox/worksheetbuffer.cxx
+++ b/sc/source/filter/oox/worksheetbuffer.cxx
@@ -44,7 +44,6 @@ using namespace ::com::sun::star::sheet;
 using namespace ::com::sun::star::uno;
 
 SheetInfoModel::SheetInfoModel() :
-    mnBiffHandle( -1 ),
     mnSheetId( -1 ),
     mnState( XML_visible )
 {
diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx
index f1ebdc5d996d..bdda325cac1c 100644
--- a/sc/source/filter/xml/xmlexprt.cxx
+++ b/sc/source/filter/xml/xmlexprt.cxx
@@ -364,8 +364,7 @@ ScXMLExport::ScXMLExport(
     nProgressCount(0),
     nCurrentTable(0),
     bHasRowHeader(false),
-    bRowHeaderOpen(false),
-    mbShowProgress( false )
+    bRowHeaderOpen(false)
 {
     if (getExportFlags() & SvXMLExportFlags::CONTENT)
     {
diff --git a/sc/source/filter/xml/xmlexprt.hxx b/sc/source/filter/xml/xmlexprt.hxx
index 2d91646b0925..2d7e3f5d510c 100644
--- a/sc/source/filter/xml/xmlexprt.hxx
+++ b/sc/source/filter/xml/xmlexprt.hxx
@@ -130,7 +130,6 @@ class ScXMLExport : public SvXMLExport
     sal_uInt16                  nCurrentTable;
     bool                        bHasRowHeader;
     bool                        bRowHeaderOpen;
-    bool                        mbShowProgress;
 
     sal_Int32       GetNumberFormatStyleIndex(sal_Int32 nNumFmt) const;
     void            CollectSharedData(SCTAB& nTableCount, sal_Int32& nShapesCount);
diff --git a/sc/source/filter/xml/xmlimprt.cxx b/sc/source/filter/xml/xmlimprt.cxx
index 47978210680d..f62f350a69ee 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -3094,11 +3094,10 @@ class RangeNameInserter : public ::std::unary_function<ScMyNamedExpression, void
 {
     ScDocument* mpDoc;
     ScRangeName& mrRangeName;
-    ScXMLImport& mrXmlImport;
 
 public:
-    RangeNameInserter(ScDocument* pDoc, ScRangeName& rRangeName, ScXMLImport& rXmlImport) :
-        mpDoc(pDoc), mrRangeName(rRangeName), mrXmlImport(rXmlImport) {}
+    RangeNameInserter(ScDocument* pDoc, ScRangeName& rRangeName) :
+        mpDoc(pDoc), mrRangeName(rRangeName) {}
 
     void operator() (const std::unique_ptr<ScMyNamedExpression>& p) const
     {
@@ -3148,7 +3147,7 @@ void ScXMLImport::SetNamedRanges()
 
     // Insert the namedRanges
     ScRangeName* pRangeNames = pDoc->GetRangeName();
-    ::std::for_each(pNamedExpressions->begin(), pNamedExpressions->end(), RangeNameInserter(pDoc, *pRangeNames, *this));
+    ::std::for_each(pNamedExpressions->begin(), pNamedExpressions->end(), RangeNameInserter(pDoc, *pRangeNames));
 }
 
 void ScXMLImport::SetSheetNamedRanges()
@@ -3164,7 +3163,7 @@ void ScXMLImport::SetSheetNamedRanges()
             continue;
 
         const ScMyNamedExpressions& rNames = *itr.second;
-        ::std::for_each(rNames.begin(), rNames.end(), RangeNameInserter(pDoc, *pRangeNames, *this));
+        ::std::for_each(rNames.begin(), rNames.end(), RangeNameInserter(pDoc, *pRangeNames));
     }
 }
 
diff --git a/sc/source/filter/xml/xmlimprt.hxx b/sc/source/filter/xml/xmlimprt.hxx
index b87e82a3db4f..882f1f0edfb6 100644
--- a/sc/source/filter/xml/xmlimprt.hxx
+++ b/sc/source/filter/xml/xmlimprt.hxx
@@ -767,7 +767,6 @@ class SolarMutexGuard;
 
 struct tScMyCellRange
 {
-    SCTAB Sheet;
     sal_Int32 StartColumn, EndColumn;
     sal_Int32 StartRow, EndRow;
 };
@@ -836,7 +835,6 @@ class ScXMLImport: public SvXMLImport, boost::noncopyable
 
     mutable std::unique_ptr<ScXMLEditAttributeMap> mpEditAttrMap;
     ScXMLChangeTrackingImportHelper*    pChangeTrackingImportHelper;
-    std::list<SvXMLImportContext*>      aViewContextList;
     ScMyStylesImportHelper*        pStylesImportHelper;
     OUString                       sNumberFormat;
     OUString                       sLocale;
diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx
index 99a6f5178964..5181bcc33bda 100644
--- a/sc/source/ui/docshell/docsh.cxx
+++ b/sc/source/ui/docshell/docsh.cxx
@@ -2645,7 +2645,6 @@ ScDocShell::ScDocShell( const ScDocShell& rShell ) :
     nDocumentLock   ( 0 ),
     nCanUpdate (css::document::UpdateDocMode::ACCORDING_TO_CONFIG),
     pOldAutoDBRange ( nullptr ),
-    pDocHelper      ( nullptr ),
     pAutoStyleList  ( nullptr ),
     pPaintLockData  ( nullptr ),
     pSolverSaveData ( nullptr ),
@@ -2690,7 +2689,6 @@ ScDocShell::ScDocShell( const SfxModelFlags i_nSfxCreationFlags ) :
     nDocumentLock   ( 0 ),
     nCanUpdate (css::document::UpdateDocMode::ACCORDING_TO_CONFIG),
     pOldAutoDBRange ( nullptr ),
-    pDocHelper      ( nullptr ),
     pAutoStyleList  ( nullptr ),
     pPaintLockData  ( nullptr ),
     pSolverSaveData ( nullptr ),
diff --git a/sc/source/ui/docshell/documentlinkmgr.cxx b/sc/source/ui/docshell/documentlinkmgr.cxx
index 87bcae28ccc4..9609781eec55 100644
--- a/sc/source/ui/docshell/documentlinkmgr.cxx
+++ b/sc/source/ui/docshell/documentlinkmgr.cxx
@@ -33,13 +33,12 @@ namespace sc {
 
 struct DocumentLinkManagerImpl : boost::noncopyable
 {
-    ScDocument& mrDoc;
     SfxObjectShell* mpShell;
     std::unique_ptr<DataStream> mpDataStream;
     std::unique_ptr<sfx2::LinkManager> mpLinkManager;
 
-    DocumentLinkManagerImpl( ScDocument& rDoc, SfxObjectShell* pShell ) :
-        mrDoc(rDoc), mpShell(pShell), mpDataStream(nullptr), mpLinkManager(nullptr) {}
+    DocumentLinkManagerImpl( SfxObjectShell* pShell ) :
+        mpShell(pShell), mpDataStream(nullptr), mpLinkManager(nullptr) {}
 
     ~DocumentLinkManagerImpl()
     {
@@ -56,8 +55,8 @@ struct DocumentLinkManagerImpl : boost::noncopyable
     }
 };
 
-DocumentLinkManager::DocumentLinkManager( ScDocument& rDoc, SfxObjectShell* pShell ) :
-    mpImpl(new DocumentLinkManagerImpl(rDoc, pShell)) {}
+DocumentLinkManager::DocumentLinkManager( SfxObjectShell* pShell ) :
+    mpImpl(new DocumentLinkManagerImpl(pShell)) {}
 
 DocumentLinkManager::~DocumentLinkManager()
 {
diff --git a/sc/source/ui/inc/docsh.hxx b/sc/source/ui/inc/docsh.hxx
index 1d1edfd3dfd5..0c0ba7d02dbb 100644
--- a/sc/source/ui/inc/docsh.hxx
+++ b/sc/source/ui/inc/docsh.hxx
@@ -102,8 +102,6 @@ class SC_DLLPUBLIC ScDocShell: public SfxObjectShell, public SfxListener
 
     ScDBData*           pOldAutoDBRange;
 
-    ScSbxDocHelper*     pDocHelper;
-
     ScAutoStyleList*    pAutoStyleList;
     ScPaintLockData*    pPaintLockData;
     ScOptSolverSave*    pSolverSaveData;
diff --git a/sc/source/ui/inc/gridwin.hxx b/sc/source/ui/inc/gridwin.hxx
index 3af7fcbff6cc..4e0172e40181 100644
--- a/sc/source/ui/inc/gridwin.hxx
+++ b/sc/source/ui/inc/gridwin.hxx
@@ -161,9 +161,6 @@ class ScGridWindow : public vcl::Window, public DropTargetHelper, public DragSou
 
     SvtScriptType           nPageScript;
 
-    long                    nLastClickX;
-    long                    nLastClickY;
-
     SCCOL                   nDragStartX;
     SCROW                   nDragStartY;
     SCCOL                   nDragEndX;
diff --git a/sc/source/ui/inc/retypepassdlg.hxx b/sc/source/ui/inc/retypepassdlg.hxx
index 03b1882648ff..c5606babd04e 100644
--- a/sc/source/ui/inc/retypepassdlg.hxx
+++ b/sc/source/ui/inc/retypepassdlg.hxx
@@ -83,7 +83,6 @@ private:
     OUString        maTextNotPassProtected;
     OUString        maTextHashBad;
     OUString        maTextHashGood;
-    OUString        maTextHashRegen;
 
     DECL_LINK_TYPED( OKHdl, Button*, void );
     DECL_LINK_TYPED( RetypeBtnHdl, Button*, void );
@@ -96,7 +95,6 @@ private:
     ::std::vector<TableItem> maTableItems;
 
     DocProtectionPtr    mpDocItem;
-    size_t              mnCurScrollPos;
     ScPasswordHash      meDesiredHash;
 };
 
diff --git a/sc/source/ui/inc/tabvwsh.hxx b/sc/source/ui/inc/tabvwsh.hxx
index 24c9c83fd20e..cb1819b4344a 100644
--- a/sc/source/ui/inc/tabvwsh.hxx
+++ b/sc/source/ui/inc/tabvwsh.hxx
@@ -95,9 +95,8 @@ class SC_DLLPUBLIC ScTabViewShell: public SfxViewShell, public ScDBFunc
 private:
     SvxHtmlOptions          aHTMLOpt;
     ObjectSelectionType     eCurOST;
-    sal_uInt16                  nDrawSfxId;
-    sal_uInt16                  nCtrlSfxId;
-    sal_uInt16                  nFormSfxId;
+    sal_uInt16              nDrawSfxId;
+    sal_uInt16              nFormSfxId;
     OUString                sDrawCustom;                // current custom shape type
     ScDrawShell*            pDrawShell;
     ScDrawTextObjectBar*    pDrawTextShell;
@@ -149,18 +148,14 @@ private:
 
     bool                    bDontSwitch;                // Don't turn off EditShell
     bool                    bInFormatDialog;            // for GetSelectionText
-    bool                    bPrintSelected;             // for result of SvxPrtQryBox
 
     bool                    bReadOnly;                  // to detect status changes
 
     bool                    bIsActive;
 
-    SbxObject*              pScSbxObject;
-
     bool                    bChartAreaValid; // if chart is drawn
     bool                    bForceFocusOnCurCell; // #i123629#
 
-    OUString                aEditChartName;
     ScRangeListRef          aChartSource;
     Rectangle               aChartPos;
     SCTAB                   nChartDestTab;
diff --git a/sc/source/ui/miscdlgs/retypepassdlg.cxx b/sc/source/ui/miscdlgs/retypepassdlg.cxx
index e7ccc2d025b0..8730f29fc13c 100644
--- a/sc/source/ui/miscdlgs/retypepassdlg.cxx
+++ b/sc/source/ui/miscdlgs/retypepassdlg.cxx
@@ -32,10 +32,8 @@ ScRetypePassDlg::ScRetypePassDlg(vcl::Window* pParent) :
     maTextNotPassProtected(ScResId(STR_NOT_PASS_PROTECTED)),
     maTextHashBad(ScResId(STR_HASH_BAD)),
     maTextHashGood(ScResId(STR_HASH_GOOD)),
-    maTextHashRegen(ScResId(STR_HASH_REGENERATED)),
 
     mpDocItem(static_cast<ScDocProtection*>(nullptr)),
-    mnCurScrollPos(0),
     meDesiredHash(PASSHASH_SHA1)
 {
     get(mpBtnOk ,"ok");
diff --git a/sc/source/ui/optdlg/calcoptionsdlg.hxx b/sc/source/ui/optdlg/calcoptionsdlg.hxx
index 1d062510a333..c7f2ffc0fb7a 100644
--- a/sc/source/ui/optdlg/calcoptionsdlg.hxx
+++ b/sc/source/ui/optdlg/calcoptionsdlg.hxx
@@ -79,9 +79,6 @@ private:
     OUString maSoftware;
 
     ScCalcConfig maConfig;
-#if HAVE_FEATURE_OPENCL
-    std::vector<OpenCLPlatformInfo> maPlatformInfo;
-#endif
 
     bool mbSelectedEmptyStringAsZero;
     bool mbWriteConfig;
diff --git a/sc/source/ui/vba/vbaapplication.cxx b/sc/source/ui/vba/vbaapplication.cxx
index d60d9378017b..a57f136500ef 100644
--- a/sc/source/ui/vba/vbaapplication.cxx
+++ b/sc/source/ui/vba/vbaapplication.cxx
@@ -101,7 +101,6 @@ using ::com::sun::star::uno::UNO_QUERY;
 /** Global application settings shared by all open workbooks. */
 struct ScVbaAppSettings
 {
-    sal_Int32 mnCalculation;
     bool mbDisplayAlerts;
     bool mbEnableEvents;
     bool mbExcel4Menus;
@@ -112,7 +111,6 @@ struct ScVbaAppSettings
 };
 
 ScVbaAppSettings::ScVbaAppSettings() :
-    mnCalculation( excel::XlCalculation::xlCalculationAutomatic ),
     mbDisplayAlerts( true ),
     mbEnableEvents( true ),
     mbExcel4Menus( false ),
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 5eca603ca8c0..e2d8ff073a75 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -476,8 +476,6 @@ ScGridWindow::ScGridWindow( vcl::Window* pParent, ScViewData* pData, ScSplitPos
             nPagebreakBreak( 0 ),
             nPagebreakPrev( 0 ),
             nPageScript( SvtScriptType::NONE ),
-            nLastClickX( 0 ),
-            nLastClickY( 0 ),
             nDragStartX( -1 ),
             nDragStartY( -1 ),
             nDragEndX( -1 ),
diff --git a/sc/source/ui/view/tabvwsh4.cxx b/sc/source/ui/view/tabvwsh4.cxx
index faa12b474655..5a6413e90313 100644
--- a/sc/source/ui/view/tabvwsh4.cxx
+++ b/sc/source/ui/view/tabvwsh4.cxx
@@ -1632,8 +1632,6 @@ ScTabViewShell::ScTabViewShell( SfxViewFrame* pViewFrame,
     ScDBFunc( &pViewFrame->GetWindow(), static_cast<ScDocShell&>(*pViewFrame->GetObjectShell()), this ),
     eCurOST(OST_NONE),
     nDrawSfxId(0),
-    nCtrlSfxId(USHRT_MAX),
-    nFormSfxId(USHRT_MAX),
     pDrawShell(nullptr),
     pDrawTextShell(nullptr),
     pEditShell(nullptr),
@@ -1667,9 +1665,7 @@ ScTabViewShell::ScTabViewShell( SfxViewFrame* pViewFrame,
     bFormShellAtTop(false),
     bDontSwitch(false),
     bInFormatDialog(false),
-    bPrintSelected(false),
     bReadOnly(false),
-    pScSbxObject(nullptr),
     bChartAreaValid(false),
     bForceFocusOnCurCell(false),
     nCurRefDlgId(0),