https://bugs.documentfoundation.org/show_bug.cgi?id=135168

--- Comment #5 from Telesto <tele...@surfxs.nl> ---
Created attachment 164177
  --> https://bugs.documentfoundation.org/attachment.cgi?id=164177&action=edit
Bibisect log

Bisected to
author  Michael Stahl <michael.st...@cib.de>    2020-04-02 17:18:37 +0200
committer       Michael Stahl <michael.st...@cib.de>    2020-04-03 17:20:22
+0200
commit  166b5010b402a41b192b1659093a25acf9065fd9 (patch)
tree    58a783dfc1800c604979380c121337ada3e5ad6f
parent  27aa4b16bf704d0246595750daf57b57ff2577b3 (diff)
tdf#130685 sw_redlinehide: fix copying to position following redline
In DocumentContentOperationsManager::CopyWithFlyInFly(), first
CopyNodes() also creates all layout frames, then SaveRedlEndPosForRestore
fixes the end position of all redlines that were moved by CopyNodes()
(they were moved not by changing their position but by inserting
new nodes before their end position).

Of course this means that the layout frames are created with redlines
that have only a temporary end position, and then things go wrong when
the end positions are adjusted, so add something similar to
SwUndoDelete::UndoImpl() to recreate the frames in CopyWithFlyInFly().

This hit the assert:
sw/source/core/text/redlnitr.cxx:94: std::unique_ptr<sw::MergedPara>
sw::CheckParaRedlineMerge(SwTextFrame&, SwTextNode&, sw::FrameMode): Assertion
`pNode != &rTextNode || &pStart->nNode.GetNode() == &rTextNode' failed.

(regression from ... sw_redlinehide)

https://cgit.freedesktop.org/libreoffice/core/commit/?id=166b5010b402a41b192b1659093a25acf9065fd9

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to