Kaydet (Commit) 3dffe225 authored tarafından Noel Grandin's avatar Noel Grandin

sal_Bool->bool and remove unnecessary param on ScConsData::InitData

since it was only called from one place, and that place
unconditionally passed "false"

Change-Id: I1b730cfff23da08a86d385737043fcabf16d37cb
üst a2c49ef9
...@@ -70,12 +70,12 @@ class ScConsData ...@@ -70,12 +70,12 @@ class ScConsData
{ {
private: private:
ScSubTotalFunc eFunction; ScSubTotalFunc eFunction;
sal_Bool bReference; bool bReference;
sal_Bool bColByName; bool bColByName;
sal_Bool bRowByName; bool bRowByName;
SCSIZE nColCount; SCSIZE nColCount;
SCSIZE nRowCount; SCSIZE nRowCount;
sal_Bool** ppUsed; bool** ppUsed;
double** ppSum; double** ppSum;
double** ppCount; double** ppCount;
double** ppSumSqr; double** ppSumSqr;
...@@ -86,7 +86,7 @@ private: ...@@ -86,7 +86,7 @@ private:
SCSIZE nTitleCount; SCSIZE nTitleCount;
OUString* mpTitles; OUString* mpTitles;
SCSIZE** ppTitlePos; SCSIZE** ppTitlePos;
sal_Bool bCornerUsed; bool bCornerUsed;
OUString aCornerText; // only for bColByName && bRowByName OUString aCornerText; // only for bColByName && bRowByName
public: public:
...@@ -94,9 +94,9 @@ public: ...@@ -94,9 +94,9 @@ public:
~ScConsData(); ~ScConsData();
void SetSize( SCCOL nCols, SCROW nRows ); void SetSize( SCCOL nCols, SCROW nRows );
void SetFlags( ScSubTotalFunc eFunc, sal_Bool bColName, sal_Bool bRowName, sal_Bool bRef ); void SetFlags( ScSubTotalFunc eFunc, bool bColName, bool bRowName, bool bRef );
void InitData(sal_Bool bDelete=sal_True); void InitData();
void DeleteData(); void DeleteData();
void AddFields( ScDocument* pSrcDoc, SCTAB nTab, void AddFields( ScDocument* pSrcDoc, SCTAB nTab,
......
...@@ -159,11 +159,8 @@ void ScConsData::DeleteData() ...@@ -159,11 +159,8 @@ void ScConsData::DeleteData()
#undef DELETEARR #undef DELETEARR
#undef DELETESTR #undef DELETESTR
void ScConsData::InitData( sal_Bool bDelete ) void ScConsData::InitData()
{ {
if (bDelete)
DeleteData();
if (bReference && nColCount && !ppRefs) if (bReference && nColCount && !ppRefs)
{ {
ppRefs = new ScReferenceList*[nColCount]; ppRefs = new ScReferenceList*[nColCount];
...@@ -185,11 +182,11 @@ void ScConsData::InitData( sal_Bool bDelete ) ...@@ -185,11 +182,11 @@ void ScConsData::InitData( sal_Bool bDelete )
if (nColCount && !ppUsed) if (nColCount && !ppUsed)
{ {
ppUsed = new sal_Bool*[nColCount]; ppUsed = new bool*[nColCount];
for (SCSIZE i=0; i<nColCount; i++) for (SCSIZE i=0; i<nColCount; i++)
{ {
ppUsed[i] = new sal_Bool[nRowCount]; ppUsed[i] = new bool[nRowCount];
memset( ppUsed[i], 0, nRowCount * sizeof(sal_Bool) ); memset( ppUsed[i], 0, nRowCount * sizeof(bool) );
} }
} }
...@@ -208,7 +205,7 @@ void ScConsData::InitData( sal_Bool bDelete ) ...@@ -208,7 +205,7 @@ void ScConsData::InitData( sal_Bool bDelete )
void ScConsData::DoneFields() void ScConsData::DoneFields()
{ {
InitData(false); InitData();
} }
void ScConsData::SetSize( SCCOL nCols, SCROW nRows ) void ScConsData::SetSize( SCCOL nCols, SCROW nRows )
...@@ -224,7 +221,7 @@ void ScConsData::GetSize( SCCOL& rCols, SCROW& rRows ) const ...@@ -224,7 +221,7 @@ void ScConsData::GetSize( SCCOL& rCols, SCROW& rRows ) const
rRows = static_cast<SCROW>(nRowCount); rRows = static_cast<SCROW>(nRowCount);
} }
void ScConsData::SetFlags( ScSubTotalFunc eFunc, sal_Bool bColName, sal_Bool bRowName, sal_Bool bRef ) void ScConsData::SetFlags( ScSubTotalFunc eFunc, bool bColName, bool bRowName, bool bRef )
{ {
DeleteData(); DeleteData();
bReference = bRef; bReference = bRef;
...@@ -304,7 +301,7 @@ void ScConsData::AddName( const OUString& rName ) ...@@ -304,7 +301,7 @@ void ScConsData::AddName( const OUString& rName )
{ {
if (!ppUsed[nArrX][nArrY]) if (!ppUsed[nArrX][nArrY])
{ {
ppUsed[nArrX][nArrY] = sal_True; ppUsed[nArrX][nArrY] = true;
ppRefs[nArrX][nArrY].Init(); ppRefs[nArrX][nArrY].Init();
} }
ppRefs[nArrX][nArrY].SetFullSize(nMax); ppRefs[nArrX][nArrY].SetFullSize(nMax);
...@@ -504,7 +501,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab, ...@@ -504,7 +501,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab,
else else
{ {
aCornerText = aThisCorner; aCornerText = aThisCorner;
bCornerUsed = sal_True; bCornerUsed = true;
} }
} }
...@@ -585,7 +582,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab, ...@@ -585,7 +582,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab,
ppRefs[nArrX][nArrY].AddEntry( nCol, nRow, nTab ); ppRefs[nArrX][nArrY].AddEntry( nCol, nRow, nTab );
else else
{ {
ppUsed[nArrX][nArrY] = sal_True; ppUsed[nArrX][nArrY] = true;
ppRefs[nArrX][nArrY].Init(); ppRefs[nArrX][nArrY].Init();
ppRefs[nArrX][nArrY].AddEntry( nCol, nRow, nTab ); ppRefs[nArrX][nArrY].AddEntry( nCol, nRow, nTab );
} }
...@@ -600,7 +597,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab, ...@@ -600,7 +597,7 @@ void ScConsData::AddData( ScDocument* pSrcDoc, SCTAB nTab,
nVal); nVal);
else else
{ {
ppUsed[nArrX][nArrY] = sal_True; ppUsed[nArrX][nArrY] = true;
lcl_InitArray( eFunction, ppCount[nArrX][nArrY], lcl_InitArray( eFunction, ppCount[nArrX][nArrY],
ppSum[nArrX][nArrY], ppSum[nArrX][nArrY],
ppSumSqr[nArrX][nArrY], nVal ); ppSumSqr[nArrX][nArrY], nVal );
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment