diff options
author | Michael Meeks <michael.meeks@collabora.com> | 2021-01-22 21:17:25 +0000 |
---|---|---|
committer | Michael Meeks <michael.meeks@collabora.com> | 2021-01-24 03:01:59 +0100 |
commit | b35f6971561bc095965e82f230e0307f6694228b (patch) | |
tree | 218d69666fb40aaa83e998ffac336a98dd6adc57 /store | |
parent | 27cec4edf112ba65d8c9d264c00cb39e2415a8cd (diff) |
sc: GetRowForHeight performance improvement.
Instead of just skipping hidden rows, either skip or interpolate
into visible ones.
This method, and it's single caller look rather unusual to me. It is
unclear why we would want to return the results we do, and why the
one caller subtracts a row.
Some surprising proportion of tile rendering was exercising this code
path extremely slowly.
--5.94%--ScDocument::GetPrintArea
|
|--5.04%--ScDrawLayer::GetPrintArea
| ScTable::GetRowForHeight
| |
| --4.58%--ScFlatBoolRowSegments::getRangeData
| |
| --2.46%--ScFlatSegmentsImpl<bool, bool>::getRangeData
Change-Id: I75418d6af59a33b99e8bb0c374139e1a4ee6ef87
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109837
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
Reviewed-by: Ashod Nakashian <ash@collabora.com>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109848
Tested-by: Jenkins
Diffstat (limited to 'store')
0 files changed, 0 insertions, 0 deletions