diff options
author | Eike Rathke <erack@redhat.com> | 2014-01-30 19:00:51 +0100 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2014-01-30 19:14:19 +0100 |
commit | 41a859234eac6126693393ff64efc7bfcdec9c54 (patch) | |
tree | a03c6c54a87215451e7198b0da0e87960fa5d635 | |
parent | 862bec45fa81b5583e764abf845c14689b339753 (diff) |
fdo#74194 Revert "Resolves: #i121136# Improve Spreadsheet performance ..."
This reverts commit 6a62557d8632290ae7150f3922dece65f51af4bc.
pPageRows[] is a member array, so when not allocating MAXROWCOUNT
entries but nRealCnt instead it also needs some tracking mechanism to
reallocate in case a subsequent call needs more elements than the first
call.
Change-Id: Iadb7d2ceadcb8bd99d57c6188571d295c6b99fba
-rw-r--r-- | sc/source/ui/unoobj/shapeuno.cxx | 3 | ||||
-rw-r--r-- | sc/source/ui/view/printfun.cxx | 9 |
2 files changed, 4 insertions, 8 deletions
diff --git a/sc/source/ui/unoobj/shapeuno.cxx b/sc/source/ui/unoobj/shapeuno.cxx index c233d8698928..7259aae509ad 100644 --- a/sc/source/ui/unoobj/shapeuno.cxx +++ b/sc/source/ui/unoobj/shapeuno.cxx @@ -842,8 +842,7 @@ uno::Any SAL_CALL ScShapeObj::getPropertyValue( const OUString& aPropertyName ) } else { - if(!pShapePropertySet) //performance consideration - GetShapePropertySet(); + GetShapePropertySet(); if (pShapePropertySet) aAny = pShapePropertySet->getPropertyValue( aPropertyName ); } diff --git a/sc/source/ui/view/printfun.cxx b/sc/source/ui/view/printfun.cxx index 0c37a182597a..b509e2b9de63 100644 --- a/sc/source/ui/view/printfun.cxx +++ b/sc/source/ui/view/printfun.cxx @@ -2963,9 +2963,8 @@ static void lcl_SetHidden( ScDocument* pDoc, SCTAB nPrintTab, ScPageRowEntry& rP void ScPrintFunc::CalcPages() // calculates aPageRect and pages from nZoom { if (!pPageEndX) pPageEndX = new SCCOL[MAXCOL+1]; - //performance impact -// if (!pPageEndY) pPageEndY = new SCROW[MAXROW+1]; -// if (!pPageRows) pPageRows = new ScPageRowEntry[MAXROW+1]; //! count before !!!! + if (!pPageEndY) pPageEndY = new SCROW[MAXROW+1]; + if (!pPageRows) pPageRows = new ScPageRowEntry[MAXROW+1]; //! count before !!!! pDoc->SetPageSize( nPrintTab, GetDocPageSize() ); if (aAreaParam.bPrintArea) @@ -2975,9 +2974,7 @@ void ScPrintFunc::CalcPages() // calculates aPageRect and pages fr } else pDoc->UpdatePageBreaks( nPrintTab, NULL ); // else, end is marked - SCROW nRealCnt = nEndRow-nStartRow+1; - if (!pPageEndY) pPageEndY = new SCROW[nRealCnt+1]; - if (!pPageRows) pPageRows = new ScPageRowEntry[nRealCnt+1]; //! vorher zaehlen !!!! + // // Page alignment/splitting after breaks in Col/RowFlags // Of several breaks in a hidden area, only one counts. |