diff options
| author | Noel Grandin <noel@peralex.com> | 2021-04-30 13:00:35 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-04-30 16:03:35 +0200 |
| commit | 857caa5fc69b92e781457a1b67a89aa051c2d70f (patch) | |
| tree | 7437d7f6345f9dea81f4203517549c48fa971eae /sfx2/source/doc/objxtor.cxx | |
| parent | b9097800f4f997de2325bc9e588e6caea7a563c7 (diff) | |
tdf#79049 speed up OOXML workbook load
we spend a lot of time in ScAttrArray::GetLastVisibleAttr
which appears to be very expensive for this worksheet.
This is re-computed every time we enter SfxBaseModel::getArgs
Reduce the recomputation by introducing a new method which
only retrieves specific SfxBaseModel arguments.
This takes the load time from 5m9 to 1m9 for me.
Change-Id: I605fae0faa94760c7d6993877c9559ea5dc813cd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114905
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2/source/doc/objxtor.cxx')
| -rw-r--r-- | sfx2/source/doc/objxtor.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sfx2/source/doc/objxtor.cxx b/sfx2/source/doc/objxtor.cxx index 6c6c7852c8d2..70b2e1e2e486 100644 --- a/sfx2/source/doc/objxtor.cxx +++ b/sfx2/source/doc/objxtor.cxx @@ -827,7 +827,7 @@ uno::Sequence< OUString > SfxObjectShell::GetEventNames() } -css::uno::Reference< css::frame::XModel > SfxObjectShell::GetModel() const +css::uno::Reference< css::frame::XModel3 > SfxObjectShell::GetModel() const { return GetBaseModel(); } @@ -843,7 +843,7 @@ void SfxObjectShell::SetBaseModel( SfxBaseModel* pModel ) } -css::uno::Reference< css::frame::XModel > SfxObjectShell::GetBaseModel() const +css::uno::Reference< css::frame::XModel3 > SfxObjectShell::GetBaseModel() const { return pImpl->pBaseModel; } |