writerfilter/source/dmapper/DomainMapper_Impl.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 412c7e98d3f0b5b03494a7618bed568f6ccf7169
Author: Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Sun Jul 21 21:05:04 2024 +0200
Commit: Miklos Vajna <vmik...@collabora.com>
CommitDate: Mon Jul 29 09:06:58 2024 +0200
drawpage::getCount is expensive
so only call it once at the start of the loop. Shaves 1% off the cost of
loading complex documents with lots of shapes.
Change-Id: Ieb985b2523f6b4db3313eb000cf64e9c689e65c3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170820
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
(cherry picked from commit 041b3d8166fcfe38aa04c39f94c2ada140eaf991)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170741
Reviewed-by: Miklos Vajna <vmik...@collabora.com>
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index 4151e51533a4..e45473634222 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -3805,7 +3805,7 @@ bool isContentEmpty(uno::Reference<text::XText> const&
xText, uno::Reference<tex
auto xDrawPage = xDrawPageSupplier->getDrawPage();
if (xDrawPage && xDrawPage->hasElements())
{
- for (sal_Int32 i = 0; i < xDrawPage->getCount(); ++i)
+ for (sal_Int32 i = 0, nCount = xDrawPage->getCount(); i < nCount; ++i)
{
uno::Reference<text::XTextContent>
xShape(xDrawPage->getByIndex(i), uno::UNO_QUERY);
if (xShape.is())
