sc/inc/document.hxx | 2 +- sc/source/core/data/documen2.cxx | 2 +- sc/source/core/data/documen4.cxx | 8 ++++---- sc/source/ui/docshell/dbdocfun.cxx | 2 +- sc/source/ui/docshell/docfunc.cxx | 4 ++-- sc/source/ui/undo/undodat.cxx | 2 +- sc/source/ui/undo/undosort.cxx | 2 +- sc/source/ui/view/dbfunc3.cxx | 2 +- 8 files changed, 12 insertions(+), 12 deletions(-)
New commits: commit 135b8d6466ade1e8724e604f705f95cecf4c4881 Author: Eike Rathke <er...@redhat.com> Date: Wed Dec 3 17:47:35 2014 +0100 remove boolean parameter default from ScDocument::SetDirty ... and force the caller to make a decision. In fact some needed to include empty cells. Change-Id: I82b6ef72f7ecda6543b3cafc70f475519870b471 diff --git a/sc/inc/document.hxx b/sc/inc/document.hxx index afa215d..41e1093 100644 --- a/sc/inc/document.hxx +++ b/sc/inc/document.hxx @@ -1012,7 +1012,7 @@ public: void ResetChanged( const ScRange& rRange ); void SetAllFormulasDirty( const sc::SetFormulaDirtyContext& rCxt ); - void SetDirty( const ScRange&, bool bIncludeEmptyCells = false ); + void SetDirty( const ScRange&, bool bIncludeEmptyCells ); void SetTableOpDirty( const ScRange& ); // for Interpreter TableOp void InterpretDirtyCells( const ScRangeList& rRanges ); SC_DLLPUBLIC void CalcAll(); diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx index bfa78cb..404dd31 100644 --- a/sc/source/core/data/documen2.cxx +++ b/sc/source/core/data/documen2.cxx @@ -1004,7 +1004,7 @@ sal_uLong ScDocument::TransferTab( ScDocument* pSrcDoc, SCTAB nSrcPos, sc::StartListeningContext aSLCxt(*this); maTabs[nDestPos]->StartListeners(aSLCxt, true); } - SetDirty( ScRange( 0, 0, nDestPos, MAXCOL, MAXROW, nDestPos)); + SetDirty( ScRange( 0, 0, nDestPos, MAXCOL, MAXROW, nDestPos), false); if ( bResultsOnly ) pSrcDoc->SetAutoCalc( bOldAutoCalcSrc ); diff --git a/sc/source/core/data/documen4.cxx b/sc/source/core/data/documen4.cxx index 5f11898..b29b1b6 100644 --- a/sc/source/core/data/documen4.cxx +++ b/sc/source/core/data/documen4.cxx @@ -126,7 +126,7 @@ bool ScDocument::Solver(SCCOL nFCol, SCROW nFRow, SCTAB nFTab, while ( !bDoneIteration && ( nIter++ < nMaxIter ) ) { *pVCell = fX; - SetDirty( aVRange ); + SetDirty( aVRange, false ); pFormula->Interpret(); bError = ( pFormula->GetErrCode() != 0 ); fF = pFormula->GetValue() - fTargetVal; @@ -160,7 +160,7 @@ bool ScDocument::Solver(SCCOL nFCol, SCROW nFRow, SCTAB nFTab, fHorX = fX - fabs( fF ) * fHorTangent; *pVCell = fHorX; - SetDirty( aVRange ); + SetDirty( aVRange, false ); pFormula->Interpret(); bHorMoveError = ( pFormula->GetErrCode() != 0 ); if ( bHorMoveError ) @@ -224,7 +224,7 @@ bool ScDocument::Solver(SCCOL nFCol, SCROW nFRow, SCTAB nFTab, if ( bDoneIteration ) { *pVCell = nX; - SetDirty( aVRange ); + SetDirty( aVRange, false ); pFormula->Interpret(); if ( fabs( pFormula->GetValue() - fTargetVal ) > fabs( fF ) ) nX = fBestX; @@ -235,7 +235,7 @@ bool ScDocument::Solver(SCCOL nFCol, SCROW nFRow, SCTAB nFTab, nX = fBestX; } *pVCell = fSaveVal; - SetDirty( aVRange ); + SetDirty( aVRange, false ); pFormula->Interpret(); if ( !bDoneIteration ) { diff --git a/sc/source/ui/docshell/dbdocfun.cxx b/sc/source/ui/docshell/dbdocfun.cxx index 72a11a3..fe254a2 100644 --- a/sc/source/ui/docshell/dbdocfun.cxx +++ b/sc/source/ui/docshell/dbdocfun.cxx @@ -1065,7 +1065,7 @@ bool ScDBDocFunc::DoSubTotals( SCTAB nTab, const ScSubTotalParam& rParam, } ScRange aDirtyRange( aNewParam.nCol1, aNewParam.nRow1, nTab, aNewParam.nCol2, aNewParam.nRow2, nTab ); - rDoc.SetDirty( aDirtyRange ); + rDoc.SetDirty( aDirtyRange, true ); if (bRecord) { diff --git a/sc/source/ui/docshell/docfunc.cxx b/sc/source/ui/docshell/docfunc.cxx index 9cd90e8..ae2d64c 100644 --- a/sc/source/ui/docshell/docfunc.cxx +++ b/sc/source/ui/docshell/docfunc.cxx @@ -4148,7 +4148,7 @@ bool ScDocFunc::TabOp( const ScRange& rRange, const ScMarkData* pTabMark, if ( aTester.IsEditable() ) { WaitObject aWait( rDocShell.GetActiveDialogParent() ); - rDoc.SetDirty( rRange ); + rDoc.SetDirty( rRange, false ); if ( bRecord ) { //! auch bei Undo selektierte Tabellen beruecksichtigen @@ -4717,7 +4717,7 @@ bool ScDocFunc::MergeCells( const ScCellMergeOption& rOption, bool bContents, bo if (bNeedContents || rOption.mbCenter) { ScRange aRange(nStartCol, nStartRow, nTab, nEndCol, nEndRow, nTab); - rDoc.SetDirty(aRange); + rDoc.SetDirty(aRange, true); } bNeedContentsUndo |= bNeedContents; diff --git a/sc/source/ui/undo/undodat.cxx b/sc/source/ui/undo/undodat.cxx index ddd57d1..980fd7b 100644 --- a/sc/source/ui/undo/undodat.cxx +++ b/sc/source/ui/undo/undodat.cxx @@ -845,7 +845,7 @@ void ScUndoQuery::Undo() ScRange aDirtyRange( 0 , aQueryParam.nRow1, nTab, MAXCOL, aQueryParam.nRow2, nTab ); - rDoc.SetDirty( aDirtyRange ); + rDoc.SetDirty( aDirtyRange, true ); DoSdrUndoAction( pDrawUndo, &rDoc ); diff --git a/sc/source/ui/undo/undosort.cxx b/sc/source/ui/undo/undosort.cxx index a4bd1af..c30cd56 100644 --- a/sc/source/ui/undo/undosort.cxx +++ b/sc/source/ui/undo/undosort.cxx @@ -46,7 +46,7 @@ void UndoSort::Execute( bool bUndo ) ScUndoUtil::MarkSimpleBlock(pDocShell, maParam.maSortRange); - rDoc.SetDirty(maParam.maSortRange); + rDoc.SetDirty(maParam.maSortRange, true); if (!aParam.mbUpdateRefs) rDoc.BroadcastCells(aParam.maSortRange, SC_HINT_DATACHANGED); diff --git a/sc/source/ui/view/dbfunc3.cxx b/sc/source/ui/view/dbfunc3.cxx index ba0d50d..9d739b1 100644 --- a/sc/source/ui/view/dbfunc3.cxx +++ b/sc/source/ui/view/dbfunc3.cxx @@ -491,7 +491,7 @@ void ScDBFunc::DoSubTotals( const ScSubTotalParam& rParam, bool bRecord, } ScRange aDirtyRange( aNewParam.nCol1, aNewParam.nRow1, nTab, aNewParam.nCol2, aNewParam.nRow2, nTab ); - rDoc.SetDirty( aDirtyRange ); + rDoc.SetDirty( aDirtyRange, true ); if (bRecord) { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits