diff options
| author | Michael Stahl <michael.stahl@allotropia.de> | 2023-08-09 13:42:33 +0200 |
|---|---|---|
| committer | Michael Stahl <michael.stahl@allotropia.de> | 2023-08-09 16:25:47 +0200 |
| commit | b5a41467efdce02b80ffe3824fdefe9fbc01fc95 (patch) | |
| tree | 84df84257cbd56774a44536993eadc820daea70d /sw/source/core/docnode/node.cxx | |
| parent | 1ba5f6682c2eaf46cbe6447125aaed5f87edeb34 (diff) | |
tdf#147938 sw_redlinehide: move fix into SwNodes::UndoTableOfText()
Reproducing tdf#151866 on the .odt attachment, it becomes clear that the
fix for tdf#147938 is in the wrong place.
There a node has the merge flag None although it is the last node of a
merge, because CheckParaRedlineMerge() first sets the node flags and
then calls DelFrames(), which now resets the flags.
In case there's another caller of DelFrames() that needs this, it's not
obvious, can be added later.
(regression from commit 5b55ccfd384d3c0c11e05f22bd7e956b4bc7e43a)
Change-Id: Iba0cb83260ecb18a4b96ded20a95b2b20d696fc4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155516
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'sw/source/core/docnode/node.cxx')
| -rw-r--r-- | sw/source/core/docnode/node.cxx | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/sw/source/core/docnode/node.cxx b/sw/source/core/docnode/node.cxx index 06a949609d91..64a9e86137c5 100644 --- a/sw/source/core/docnode/node.cxx +++ b/sw/source/core/docnode/node.cxx @@ -1506,15 +1506,6 @@ void SwContentNode::DelFrames(SwRootFrame const*const pLayout) pMerged->listener.EndListening(this); continue; // don't delete } - else - { - for (SwNodeOffset i = pMerged->pLastNode->GetIndex(); - GetIndex() <= i; --i) - { // tdf#147938 reset flag in nodes - SwNode *const pNode = GetNodes()[i]; - pNode->SetRedlineMergeFlag(SwNode::Merge::None); - } - } } // #i27138# // notify accessibility paragraphs objects about changed |