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) /