diff options
author | Noel Grandin <noelgrandin@gmail.com> | 2024-09-28 14:59:14 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-09-28 20:09:18 +0200 |
commit | 2264538ec245839651de4e34cec45958460c4021 (patch) | |
tree | 26f51a171ba960d33825549f3fbe81852c228d8c | |
parent | 980f28fce8ccf317ee099179b07e68a50119dc6e (diff) |
cid#1607691 Overflowed constant
Change-Id: Ib3f32065334769298483d4cf9131040ab28a3a1a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174121
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-rw-r--r-- | sw/source/core/doc/DocumentContentOperationsManager.cxx | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx b/sw/source/core/doc/DocumentContentOperationsManager.cxx index 0f24abd95bc3..79ad75ea2007 100644 --- a/sw/source/core/doc/DocumentContentOperationsManager.cxx +++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx @@ -828,7 +828,7 @@ namespace // iterate over relevant redlines and decide for each whether it should // be saved, or split + saved SwRedlineTable& rRedlineTable = rDoc.getIDocumentRedlineAccess().GetRedlineTable(); - for( ; nCurrentRedline < rRedlineTable.size(); nCurrentRedline++ ) + for( ; nCurrentRedline < rRedlineTable.size(); ) { SwRangeRedline* pCurrent = rRedlineTable[ nCurrentRedline ]; SwComparePosition eCompare = @@ -843,7 +843,7 @@ namespace eCompare == SwComparePosition::Inside || eCompare == SwComparePosition::Equal ) { - rRedlineTable.Remove( nCurrentRedline-- ); + rRedlineTable.Remove( nCurrentRedline ); // split beginning, if necessary if( eCompare == SwComparePosition::OverlapBefore || @@ -868,6 +868,8 @@ namespace // save the current redline rArr.emplace_back( pCurrent, *pStart ); } + else + nCurrentRedline++; } // restore old redline mode |