sw/source/core/crsr/crsrsh.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit e4ecee9eadb496a4635232514e024ef9d71a454f
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Thu Sep 15 10:16:38 2022 +0300
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Thu Sep 15 11:44:06 2022 +0200
Fix the obvious leftover
This was introduced this way since commit
468c7b8d3287712c6959cd531c71875f4d26bcbc
Author Jens-Heiner Rechtien <h...@openoffice.org>
Date Fri Jan 04 12:19:29 2008 +0000
INTEGRATION: CWS sw8u10bf02 (1.66.92); FILE MERGED
Obviously, since pNode was not updated, the next check would fail again,
and the updated value of aIdx would be discarded immediately, making the
first check and the call to GoNext effectively a no-op.
Change-Id: I878cd0887197a86d8800a467807275ed09e617c8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139968
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/sw/source/core/crsr/crsrsh.cxx b/sw/source/core/crsr/crsrsh.cxx
index a03a4c6d68de..c734318491c8 100644
--- a/sw/source/core/crsr/crsrsh.cxx
+++ b/sw/source/core/crsr/crsrsh.cxx
@@ -3640,7 +3640,7 @@ void SwCursorShell::ClearUpCursors()
SwNodeIndex aIdx( pStartCursor->GetPoint()->GetNode() );
SwNode * pNode = SwNodes::GoPrevious(&aIdx);
if( pNode == nullptr || lcl_NodeContext( *pNode ) != pStart )
- aNodes.GoNext( &aIdx );
+ pNode = aNodes.GoNext( &aIdx );
if( pNode == nullptr || lcl_NodeContext( *pNode ) != pStart )
{
// If the start entry of the ring is invalid replace it with a
