sw/source/filter/xml/xmltbli.cxx |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit 0875594b381e2765f2b6413074c470f9eec33201
Author:     Julien Nabet <serval2...@yahoo.fr>
AuthorDate: Tue Oct 17 15:10:00 2023 +0200
Commit:     Julien Nabet <serval2...@yahoo.fr>
CommitDate: Tue Oct 17 16:42:33 2023 +0200

    Related tdf#157726: make iterators local to loops
    
    to avoid crash in debug mode, see bt here:
    https://bugs.documentfoundation.org/attachment.cgi?id=190245)
    
    + use != for iterator comparisons
    
    Change-Id: I5b1e502097d723e6acc17687171195d81c60dcbe
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158086
    Reviewed-by: Julien Nabet <serval2...@yahoo.fr>
    Tested-by: Julien Nabet <serval2...@yahoo.fr>

diff --git a/sw/source/filter/xml/xmltbli.cxx b/sw/source/filter/xml/xmltbli.cxx
index 1c9c6a8345b2..72b99fb0f7ec 100644
--- a/sw/source/filter/xml/xmltbli.cxx
+++ b/sw/source/filter/xml/xmltbli.cxx
@@ -2218,7 +2218,6 @@ SwTableLine *SwXMLTableContext::MakeTableLine( SwTableBox 
*pUpper,
 void SwXMLTableContext::MakeTable_( SwTableBox *pBox )
 {
     // fix column widths
-    std::vector<ColumnWidthInfo>::iterator colIter;
     sal_uInt32 nCols = GetColumnCount();
 
     // If there are empty rows (because of some row span of previous rows)
@@ -2316,7 +2315,7 @@ void SwXMLTableContext::MakeTable_( SwTableBox *pBox )
         {
             double n = static_cast<double>(m_nWidth) / 
static_cast<double>(nRelWidth);
             nRelWidth = 0;
-            for( colIter = m_aColumnWidths.begin(); colIter < 
m_aColumnWidths.end() - 1; ++colIter)
+            for( auto colIter = m_aColumnWidths.begin(); colIter != 
(m_aColumnWidths.end() - 1); ++colIter)
             {
                 sal_Int32 nW = static_cast<sal_Int32>( colIter->width * n);
                 colIter->width = o3tl::narrowing<sal_uInt16>(nW);
@@ -2417,7 +2416,7 @@ void SwXMLTableContext::MakeTable_( SwTableBox *pBox )
                 // column widths, every column get some extra width.
                 sal_Int32 nExtraAbs = m_nWidth - nAbsWidth;
                 sal_Int32 nAbsLastCol = m_aColumnWidths.back().width + 
nExtraAbs;
-                for( colIter = m_aColumnWidths.begin(); colIter < 
m_aColumnWidths.end()-1; ++colIter )
+                for( auto colIter = m_aColumnWidths.begin(); colIter != 
(m_aColumnWidths.end() - 1); ++colIter )
                 {
                     sal_Int32 nAbsCol = colIter->width;
                     sal_Int32 nExtraAbsCol = (nAbsCol * nExtraAbs) /
@@ -2435,7 +2434,7 @@ void SwXMLTableContext::MakeTable_( SwTableBox *pBox )
                 // Every column gets the minimum width plus some extra width.
                 sal_Int32 nExtraAbs = m_nWidth - (nCols * MINLAY);
                 sal_Int32 nAbsLastCol = MINLAY + nExtraAbs;
-                for( colIter = m_aColumnWidths.begin(); colIter < 
m_aColumnWidths.end()-1; ++colIter )
+                for( auto colIter = m_aColumnWidths.begin(); colIter != 
(m_aColumnWidths.end() - 1); ++colIter )
                 {
                     sal_Int32 nAbsCol = colIter->width;
                     sal_Int32 nExtraAbsCol = (nAbsCol * nExtraAbs) /

Reply via email to