diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-01-16 12:10:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-01-17 07:00:03 +0100 |
| commit | 3596c9891e16e1222208b18bdcdc9909c2f02d0f (patch) | |
| tree | 22983a9fc6ec7f455236d27e25562c098329e39e /sc/source/core/data/fillinfo.cxx | |
| parent | 6095612850973388ba5b121b34d02292a2548e7d (diff) | |
use std::vector in ScMarkArray, instead of re-inventing the wheel
and compact the ScMarkEntry record, this stuff shows up on perf profiles
Also make these classes so we don't need to #include the cxx into a
unit test.
Change-Id: Id806385ae877a576ec25e7772c972448dada130b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86907
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/core/data/fillinfo.cxx')
| -rw-r--r-- | sc/source/core/data/fillinfo.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/source/core/data/fillinfo.cxx b/sc/source/core/data/fillinfo.cxx index cf649db4b5eb..747d69b91fbb 100644 --- a/sc/source/core/data/fillinfo.cxx +++ b/sc/source/core/data/fillinfo.cxx @@ -608,7 +608,7 @@ void ScDocument::FillInfo( { do { - nThisRow=aThisMarkArr.pData[nIndex].nRow; // End of range + nThisRow=aThisMarkArr.mvData[nIndex].nRow; // End of range do { @@ -621,7 +621,7 @@ void ScDocument::FillInfo( while (nCurRow <= nThisRow && nCurRow <= nRow2); ++nIndex; } - while ( nIndex < aThisMarkArr.nCount && nThisRow < nRow2 ); + while ( nIndex < aThisMarkArr.mvData.size() && nThisRow < nRow2 ); } } } @@ -764,7 +764,7 @@ void ScDocument::FillInfo( ScMarkArray aThisMarkArr(pMarkData->GetMarkArray( nStartX )); SCSIZE nIndex; if ( aThisMarkArr.Search( nStartY, nIndex ) ) - bCellMarked=aThisMarkArr.pData[nIndex].bMarked; + bCellMarked=aThisMarkArr.mvData[nIndex].bMarked; } } } |