diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-09-15 12:01:50 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2020-09-16 14:22:01 +0200 |
commit | b979802fc92c8386fa35893f3c620082cec8fbde (patch) | |
tree | 6a6088a5a102c944a95ca797a25f8923358f13a3 /sc/source/core/data/documen2.cxx | |
parent | 120588850a43910fd2a6cd5ff903feece714b525 (diff) |
ScTable ctor never called with a null ScDocument*
Change-Id: I183fd26df45ddd4e7432833c5c77b07b64e03609
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102842
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sc/source/core/data/documen2.cxx')
-rw-r--r-- | sc/source/core/data/documen2.cxx | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx index 3e08ae45a35d..6015aa596b55 100644 --- a/sc/source/core/data/documen2.cxx +++ b/sc/source/core/data/documen2.cxx @@ -495,7 +495,7 @@ void ScDocument::ResetClip( ScDocument* pSourceDoc, const ScMarkData* pMarks ) OUString aString = pSourceDoc->maTabs[i]->GetName(); if ( i < static_cast<SCTAB>(maTabs.size()) ) { - maTabs[i].reset( new ScTable(this, i, aString) ); + maTabs[i].reset( new ScTable(*this, i, aString) ); } else @@ -504,7 +504,7 @@ void ScDocument::ResetClip( ScDocument* pSourceDoc, const ScMarkData* pMarks ) { maTabs.resize(i); } - maTabs.emplace_back(new ScTable(this, i, aString)); + maTabs.emplace_back(new ScTable(*this, i, aString)); } maTabs[i]->SetLayoutRTL( pSourceDoc->maTabs[i]->IsLayoutRTL() ); } @@ -524,7 +524,7 @@ void ScDocument::ResetClip( ScDocument* pSourceDoc, SCTAB nTab ) { maTabs.resize(nTab+1); } - maTabs[nTab].reset( new ScTable(this, nTab, "baeh") ); + maTabs[nTab].reset( new ScTable(*this, nTab, "baeh") ); if (nTab < static_cast<SCTAB>(pSourceDoc->maTabs.size()) && pSourceDoc->maTabs[nTab]) maTabs[nTab]->SetLayoutRTL( pSourceDoc->maTabs[nTab]->IsLayoutRTL() ); } @@ -541,7 +541,7 @@ void ScDocument::EnsureTable( SCTAB nTab ) maTabs.resize(nTab+1); if (!maTabs[nTab]) - maTabs[nTab].reset( new ScTable(this, nTab, "temp", bExtras, bExtras) ); + maTabs[nTab].reset( new ScTable(*this, nTab, "temp", bExtras, bExtras) ); } ScRefCellValue ScDocument::GetRefCellValue( const ScAddress& rPos ) @@ -790,7 +790,7 @@ bool ScDocument::CopyTab( SCTAB nOldPos, SCTAB nNewPos, const ScMarkData* pOnlyM if (nNewPos >= static_cast<SCTAB>(maTabs.size())) { nNewPos = static_cast<SCTAB>(maTabs.size()); - maTabs.emplace_back(new ScTable(this, nNewPos, aName)); + maTabs.emplace_back(new ScTable(*this, nNewPos, aName)); } else { @@ -820,7 +820,7 @@ bool ScDocument::CopyTab( SCTAB nOldPos, SCTAB nNewPos, const ScMarkData* pOnlyM (*it)->UpdateInsertTab(aCxt); if (nNewPos <= nOldPos) nOldPos++; - maTabs.emplace(maTabs.begin() + nNewPos, new ScTable(this, nNewPos, aName)); + maTabs.emplace(maTabs.begin() + nNewPos, new ScTable(*this, nNewPos, aName)); bValid = true; for (TableContainer::iterator it = maTabs.begin(); it != maTabs.end(); ++it) if (*it && it != maTabs.begin()+nOldPos && it != maTabs.begin() + nNewPos) |