sw/source/core/layout/pagechg.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit e76c7bc413af5ca1d74acb71a5a8bb6095605a89
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Sat Sep 28 20:08:38 2024 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sat Sep 28 21:25:31 2024 +0200

    Revert "cid#1607805 Overflowed constant"
    
    This reverts commit f063ea562c6ba14db15b9773bdcbb18af4a72ff8.
    
    Reason for revert: bad idea. We're not going to re-evalute the loop 
condition if we use a goto
    
    Change-Id: I8b892cd35c3560396f65619cd095a9b186ecbd24
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174011
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/sw/source/core/layout/pagechg.cxx 
b/sw/source/core/layout/pagechg.cxx
index 351ad146d6e0..d086166caaef 100644
--- a/sw/source/core/layout/pagechg.cxx
+++ b/sw/source/core/layout/pagechg.cxx
@@ -411,7 +411,6 @@ static void lcl_MakeObjs(const 
sw::FrameFormats<sw::SpzFrameFormat*>& rSpzs, SwP
     // formats are in the special table of the document
     for(size_t i = 0; i < rSpzs.size(); ++i )
     {
-        LOOP_START:
         auto pSpz = rSpzs[i];
         const SwFormatAnchor &rAnch = pSpz->GetAnchor();
         if ( rAnch.GetPageNum() == pPage->GetPhyPageNum() )
@@ -435,7 +434,8 @@ static void lcl_MakeObjs(const 
sw::FrameFormats<sw::SpzFrameFormat*>& rSpzs, SwP
             {
                 OSL_FAIL( "DrawObject not found." );
                 pSpz->GetDoc()->DelFrameFormat( pSpz );
-                goto LOOP_START;
+                --i;
+                continue;
             }
             // The object might be anchored to another page, e.g. when 
inserting
             // a new page due to a page descriptor change. In such cases, the

Reply via email to