diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-13 17:02:09 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-15 07:30:26 +0100 |
commit | a3aceef8200e48173df4b5e67b5d6fb393a23ad9 (patch) | |
tree | 7b0b0b117c8e3c15c2b126b085d11a21c2446991 /sc/source/filter/excel | |
parent | d775ef360168271f429466bbc174ae7dec402f1d (diff) |
don't use heap for elements in ScRangePairList
no need to store small objects like this out of line.
And use std::array for ScRangePair, so we get range checking in debug
mode.
Change-Id: Ie4690edbb4c3fdc4e08206cb016b8167a399d95a
Reviewed-on: https://gerrit.libreoffice.org/51268
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/filter/excel')
-rw-r--r-- | sc/source/filter/excel/xecontent.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sc/source/filter/excel/xecontent.cxx b/sc/source/filter/excel/xecontent.cxx index 473200a13c5a..8cd7de4b5066 100644 --- a/sc/source/filter/excel/xecontent.cxx +++ b/sc/source/filter/excel/xecontent.cxx @@ -551,8 +551,8 @@ void XclExpLabelranges::FillRangeList( ScRangeList& rScRanges, { for ( size_t i = 0, nPairs = xLabelRangesRef->size(); i < nPairs; ++i ) { - ScRangePair* pRangePair = (*xLabelRangesRef)[i]; - const ScRange& rScRange = pRangePair->GetRange( 0 ); + const ScRangePair & rRangePair = (*xLabelRangesRef)[i]; + const ScRange& rScRange = rRangePair.GetRange( 0 ); if( rScRange.aStart.Tab() == nScTab ) rScRanges.push_back( rScRange ); } |