[Libreoffice-commits] core.git: svx/source

[Noel Grandin (via logerrit)]

 svx/source/svdraw/svdpage.cxx |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit 75bff1d686307a4c17e1ffa949201f28028f0948
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Tue Jun 28 12:30:01 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Tue Jun 28 14:39:17 2022 +0200

    tdf#134328 improve SdrObjList::RecalcObjOrdNums
    
    rather iterate directly over the data structure, cheaper than repeatedly
    using operator[]
    
    Change-Id: Ib688ba76e9af01dbaad8ce7fc22d53fa8886f97d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136568
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/svx/source/svdraw/svdpage.cxx b/svx/source/svdraw/svdpage.cxx
index b7aa8777e39c..a98b88947b76 100644
--- a/svx/source/svdraw/svdpage.cxx
+++ b/svx/source/svdraw/svdpage.cxx
@@ -229,11 +229,9 @@ void SdrObjList::CopyObjects(const SdrObjList& rSrcList)
 
 void SdrObjList::RecalcObjOrdNums()
 {
-    const size_t nCount = GetObjCount();
-    for (size_t no=0; no<nCount; ++no) {
-        SdrObject* pObj=GetObj(no);
-        pObj->SetOrdNum(no);
-    }
+    size_t no=0;
+    for (SdrObject* pObj : maList)
+        pObj->SetOrdNum(no++);
     mbObjOrdNumsDirty=false;
 }