[Libreoffice-commits] core.git: Branch 'feature/fixes10' - sc/inc sc/source
László Németh
laszlo.nemeth at collabora.com
Tue Sep 15 08:38:21 PDT 2015
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.nemeth at 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);
More information about the Libreoffice-commits
mailing list