From ad067c367fbca9eb7a0f5c8adbe6be243950569e Mon Sep 17 00:00:00 2001 From: Michael Stahl Date: Wed, 8 Aug 2018 16:13:58 +0200 Subject: sw_redlinehide_2: fix ordering of SplitNode usage of ContentIdxStore The problem is that now the ctor of SwTextFrame will check the redline positions, but the call to MakeFramesForAdjacentContentNode() happens before the call to ContentIdxStore::Restore() that updates the SwPositions of the redlines, hence they point to the wrong node. Try to fix this by not calling Restore directly but pass in a closure to SwTextNode::SplitContentNode() so that it can call ContentIdxStore::Restore() before frames are created and redline positions are checked. Also remove the useless SwContentNode::SplitContentNode() - only the SwTextNode override actually did anything. Change-Id: I2088fd124d04cf354f4f0f691a50ff5217d778d7 --- sw/inc/ndtxt.hxx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'sw/inc/ndtxt.hxx') diff --git a/sw/inc/ndtxt.hxx b/sw/inc/ndtxt.hxx index c9e1f2b334ae..98f1af0de448 100644 --- a/sw/inc/ndtxt.hxx +++ b/sw/inc/ndtxt.hxx @@ -34,6 +34,7 @@ #include #include #include +#include class SfxHint; class SwNumRule; @@ -347,7 +348,8 @@ public: /// Virtual methods from ContentNode. virtual SwContentFrame *MakeFrame( SwFrame* ) override; - virtual SwContentNode *SplitContentNode( const SwPosition & ) override; + SwTextNode * SplitContentNode(const SwPosition &, + std::function const* pContentIndexRestore); virtual SwContentNode *JoinNext() override; void JoinPrev(); -- cgit