sw/source/core/doc/DocumentRedlineManager.cxx | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-)
New commits: commit cc2babfa0a960c7d52ea7997aea19dcf10c12d08 Author: Miklos Vajna <vmik...@collabora.com> AuthorDate: Tue Jun 3 16:57:27 2025 +0200 Commit: Miklos Vajna <vmik...@collabora.com> CommitDate: Wed Jun 4 15:23:01 2025 +0200 sw interdependent redlines: fix nPamEndtNI -> nPamEndNI typo That extra 't' doesn't seem to have any meaning and is inconsistent with the content index & start node index naming. Change-Id: Ibd938aeb91ffa281f028d5d10309a1e0fda7c541 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186190 Reviewed-by: Miklos Vajna <vmik...@collabora.com> Tested-by: Jenkins diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx index 0614840f6027..22547274d414 100644 --- a/sw/source/core/doc/DocumentRedlineManager.cxx +++ b/sw/source/core/doc/DocumentRedlineManager.cxx @@ -3318,7 +3318,7 @@ bool DocumentRedlineManager::AcceptRedlineRange(SwRedlineTable::size_type nPosOr SwNodeOffset nPamStartNI = maRedlineTable[nPosStart]->Start()->GetNodeIndex(); sal_Int32 nPamStartCI = maRedlineTable[nPosStart]->Start()->GetContentIndex(); - SwNodeOffset nPamEndtNI = maRedlineTable[nPosEnd]->End()->GetNodeIndex(); + SwNodeOffset nPamEndNI = maRedlineTable[nPosEnd]->End()->GetNodeIndex(); sal_Int32 nPamEndCI = maRedlineTable[nPosEnd]->End()->GetContentIndex(); do { @@ -3329,8 +3329,8 @@ bool DocumentRedlineManager::AcceptRedlineRange(SwRedlineTable::size_type nPosOr && pTmp->Start()->GetContentIndex() < nPamStartCI)) break; - if (pTmp->End()->GetNodeIndex() > nPamEndtNI - || (pTmp->End()->GetNodeIndex() == nPamEndtNI + if (pTmp->End()->GetNodeIndex() > nPamEndNI + || (pTmp->End()->GetNodeIndex() == nPamEndNI && pTmp->End()->GetContentIndex() > nPamEndCI)) { } @@ -3341,7 +3341,7 @@ bool DocumentRedlineManager::AcceptRedlineRange(SwRedlineTable::size_type nPosOr m_rDoc.GetIDocumentUndoRedo().AppendUndo( std::make_unique<SwUndoAcceptRedline>(*pTmp)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); bool bHierarchicalFormat = pTmp->GetType() == RedlineType::Format && pTmp->GetStackCount() > 1; @@ -3371,7 +3371,7 @@ bool DocumentRedlineManager::AcceptRedlineRange(SwRedlineTable::size_type nPosOr m_rDoc.GetIDocumentUndoRedo().AppendUndo( std::make_unique<SwUndoAcceptRedline>(*pTmp, 1)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); if (aOrigData.GetType() == RedlineType::Delete) { @@ -3394,7 +3394,7 @@ bool DocumentRedlineManager::AcceptRedlineRange(SwRedlineTable::size_type nPosOr m_rDoc.GetIDocumentUndoRedo().AppendUndo( std::make_unique<SwUndoAcceptRedline>(*pTmp)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); bRet |= lcl_AcceptRedline(maRedlineTable, nRdlIdx, bCallDelete); nRdlIdx++; @@ -3630,7 +3630,7 @@ bool DocumentRedlineManager::RejectRedlineRange(SwRedlineTable::size_type nPosOr SwNodeOffset nPamStartNI = maRedlineTable[nPosStart]->Start()->GetNodeIndex(); sal_Int32 nPamStartCI = maRedlineTable[nPosStart]->Start()->GetContentIndex(); - SwNodeOffset nPamEndtNI = maRedlineTable[nPosEnd]->End()->GetNodeIndex(); + SwNodeOffset nPamEndNI = maRedlineTable[nPosEnd]->End()->GetNodeIndex(); sal_Int32 nPamEndCI = maRedlineTable[nPosEnd]->End()->GetContentIndex(); do { @@ -3641,8 +3641,8 @@ bool DocumentRedlineManager::RejectRedlineRange(SwRedlineTable::size_type nPosOr && pTmp->Start()->GetContentIndex() < nPamStartCI)) break; - if (pTmp->End()->GetNodeIndex() > nPamEndtNI - || (pTmp->End()->GetNodeIndex() == nPamEndtNI + if (pTmp->End()->GetNodeIndex() > nPamEndNI + || (pTmp->End()->GetNodeIndex() == nPamEndNI && pTmp->End()->GetContentIndex() > nPamEndCI)) { } @@ -3668,7 +3668,7 @@ bool DocumentRedlineManager::RejectRedlineRange(SwRedlineTable::size_type nPosOr #endif m_rDoc.GetIDocumentUndoRedo().AppendUndo(std::move(pUndoRdl)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); if (bHierarchicalFormat && pTmp->GetType(1) == RedlineType::Insert) @@ -3704,7 +3704,7 @@ bool DocumentRedlineManager::RejectRedlineRange(SwRedlineTable::size_type nPosOr #endif m_rDoc.GetIDocumentUndoRedo().AppendUndo(std::move(pUndoRdl)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); std::optional<SwPaM> oPam; RedlineType eInnerType = aOrigData.GetType(); @@ -3745,7 +3745,7 @@ bool DocumentRedlineManager::RejectRedlineRange(SwRedlineTable::size_type nPosOr = std::make_unique<SwUndoRejectRedline>(*pTmp); m_rDoc.GetIDocumentUndoRedo().AppendUndo(std::move(pUndoRdl)); } - nPamEndtNI = pTmp->Start()->GetNodeIndex(); + nPamEndNI = pTmp->Start()->GetNodeIndex(); nPamEndCI = pTmp->Start()->GetContentIndex(); bRet |= lcl_RejectRedline(maRedlineTable, nRdlIdx, bCallDelete); nRdlIdx++;