sc/inc/formulacell.hxx | 14 +++---- sc/source/core/data/formulacell.cxx | 72 ++++++++++++++++++------------------ 2 files changed, 43 insertions(+), 43 deletions(-)
New commits: commit 750ee5aa2f5f9cef008ff076bd5a5d69948f4c91 Author: László Németh <laszlo.nem...@collabora.com> Date: Tue Sep 15 17:36:54 2015 +0200 re-ordering the ScFormulaCell structure Change-Id: Id829c193d0287949705168f86c302743474e2b03 diff --git a/sc/inc/formulacell.hxx b/sc/inc/formulacell.hxx index 66f4c9b..64ec06d 100644 --- a/sc/inc/formulacell.hxx +++ b/sc/inc/formulacell.hxx @@ -121,14 +121,7 @@ class SC_DLLPUBLIC ScFormulaCell : public SvtListener { private: ScFormulaCellGroupRef mxGroup; // re-factoring hack - group of formulae we're part of. - ScFormulaResult aResult; - formula::FormulaGrammar::Grammar eTempGrammar; // used between string (creation) and (re)compilation - ScTokenArray* pCode; // The (new) token array ScDocument* pDocument; - ScFormulaCell* pPrevious; - ScFormulaCell* pNext; - ScFormulaCell* pPreviousTrack; - ScFormulaCell* pNextTrack; sal_uInt16 nSeenInIteration; // Iteration cycle in which the cell was last encountered short nFormatType; sal_uInt8 cMatrixFlag : 2; // One of ScMatrixMode @@ -144,6 +137,13 @@ private: bool mbNeedsNumberFormat : 1; // set the calculated number format as hard number format bool mbPostponedDirty : 1; // if cell needs to be set dirty later bool mbIsExtRef : 1; // has references in ScExternalRefManager; never cleared after set + ScFormulaResult aResult; + ScTokenArray* pCode; // The (new) token array + ScFormulaCell* pPrevious; + ScFormulaCell* pNext; + ScFormulaCell* pPreviousTrack; + ScFormulaCell* pNextTrack; + formula::FormulaGrammar::Grammar eTempGrammar; // used between string (creation) and (re)compilation enum ScInterpretTailParameter { diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx index b4ba29b..b55142f 100644 --- a/sc/source/core/data/formulacell.cxx +++ b/sc/source/core/data/formulacell.cxx @@ -589,13 +589,7 @@ void ScFormulaCellGroup::endAllGroupListening( ScDocument& rDoc ) } ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos ) : - eTempGrammar(formula::FormulaGrammar::GRAM_DEFAULT), - pCode(new ScTokenArray), pDocument(pDoc), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType(css::util::NumberFormat::NUMBER), cMatrixFlag(MM_NONE), @@ -611,6 +605,12 @@ ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos ) : mbNeedsNumberFormat(false), mbPostponedDirty(false), mbIsExtRef(false), + pCode(new ScTokenArray), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar(formula::FormulaGrammar::GRAM_DEFAULT), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this); @@ -620,13 +620,7 @@ ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, const OUString& rFormula, const FormulaGrammar::Grammar eGrammar, sal_uInt8 cMatInd ) : - eTempGrammar( eGrammar), - pCode( NULL ), pDocument( pDoc ), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType ( css::util::NumberFormat::NUMBER ), cMatrixFlag ( cMatInd ), @@ -642,6 +636,12 @@ ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, mbNeedsNumberFormat( false ), mbPostponedDirty(false), mbIsExtRef(false), + pCode( NULL ), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar( eGrammar), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this); @@ -655,13 +655,7 @@ ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, ScTokenArray* pArray, const FormulaGrammar::Grammar eGrammar, sal_uInt8 cMatInd ) : - eTempGrammar( eGrammar), - pCode(pArray), pDocument( pDoc ), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType ( css::util::NumberFormat::NUMBER ), cMatrixFlag ( cMatInd ), @@ -677,6 +671,12 @@ ScFormulaCell::ScFormulaCell( mbNeedsNumberFormat( false ), mbPostponedDirty(false), mbIsExtRef(false), + pCode(pArray), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar( eGrammar), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this); @@ -705,13 +705,7 @@ ScFormulaCell::ScFormulaCell( ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, const ScTokenArray& rArray, const FormulaGrammar::Grammar eGrammar, sal_uInt8 cMatInd ) : - eTempGrammar( eGrammar), - pCode(new ScTokenArray(rArray)), pDocument( pDoc ), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType ( css::util::NumberFormat::NUMBER ), cMatrixFlag ( cMatInd ), @@ -727,6 +721,12 @@ ScFormulaCell::ScFormulaCell( mbNeedsNumberFormat( false ), mbPostponedDirty(false), mbIsExtRef(false), + pCode(new ScTokenArray(rArray)), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar( eGrammar), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this); @@ -755,13 +755,7 @@ ScFormulaCell::ScFormulaCell( ScDocument* pDoc, const ScAddress& rPos, const ScFormulaCellGroupRef& xGroup, const FormulaGrammar::Grammar eGrammar, sal_uInt8 cInd ) : mxGroup(xGroup), - eTempGrammar( eGrammar), - pCode(xGroup->mpCode ? xGroup->mpCode : new ScTokenArray), pDocument( pDoc ), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType(xGroup->mnFormatType), cMatrixFlag ( cInd ), @@ -777,6 +771,12 @@ ScFormulaCell::ScFormulaCell( mbNeedsNumberFormat( false ), mbPostponedDirty(false), mbIsExtRef(false), + pCode(xGroup->mpCode ? xGroup->mpCode : new ScTokenArray), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar( eGrammar), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this); @@ -787,13 +787,7 @@ ScFormulaCell::ScFormulaCell( ScFormulaCell::ScFormulaCell( const ScFormulaCell& rCell, ScDocument& rDoc, const ScAddress& rPos, int nCloneFlags ) : SvtListener(), - aResult( rCell.aResult ), - eTempGrammar( rCell.eTempGrammar), pDocument( &rDoc ), - pPrevious(0), - pNext(0), - pPreviousTrack(0), - pNextTrack(0), nSeenInIteration(0), nFormatType( rCell.nFormatType ), cMatrixFlag ( rCell.cMatrixFlag ), @@ -809,6 +803,12 @@ ScFormulaCell::ScFormulaCell( const ScFormulaCell& rCell, ScDocument& rDoc, cons mbNeedsNumberFormat( false ), mbPostponedDirty(false), mbIsExtRef(false), + aResult( rCell.aResult ), + pPrevious(0), + pNext(0), + pPreviousTrack(0), + pNextTrack(0), + eTempGrammar( rCell.eTempGrammar), aPos(rPos) { SAL_INFO( "sc.core.formulacell", "ScFormulaCell ctor this " << this);
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits