diff options
author | Michael Stahl <Michael.Stahl@cib.de> | 2019-06-02 13:58:01 +0200 |
---|---|---|
committer | Michael Stahl <Michael.Stahl@cib.de> | 2019-06-03 10:42:08 +0200 |
commit | 1416c5426a757d32f223cab46bf9038bd7f21d48 (patch) | |
tree | e00ffa11a79851fe51af1a0440388c7a9b7e505b /sw | |
parent | 2c2edfe3fe2c1ea692a6fa9488bbdab4e56dd8b9 (diff) |
tdf#119109 sw: tweak assertion in PrepareMake()
Empty section frames cause trouble for the assert added in
e14056e6e88d9b8d988b7b88b2776a8fc952031b:
* existing ones may be skipped by MoveBwd(), so a SwTextFrame moves to
the previous page but the SwSectionFrame that was its mpPrev remains
on this page; e.g. ooo110854-1.rtf
* a SwSectionFrame moves backward, then some SwTextFrame in it splits
and the follow tries to move forward, resulting in *new*
SwSectionFrames on this page that are empty (because the follow is
eventually joined again); e.g. abi3213-1.rtf
So if there's a section frame, assume it's SNAFU and don't assert.
Change-Id: Ibdca2aa39fae123583e5edf5173f80e8b70ef335
Reviewed-on: https://gerrit.libreoffice.org/73333
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
Diffstat (limited to 'sw')
-rw-r--r-- | sw/source/core/layout/calcmove.cxx | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sw/source/core/layout/calcmove.cxx b/sw/source/core/layout/calcmove.cxx index 051bd6ca5a9c..fc4bc4567079 100644 --- a/sw/source/core/layout/calcmove.cxx +++ b/sw/source/core/layout/calcmove.cxx @@ -319,6 +319,8 @@ void SwFrame::PrepareMake(vcl::RenderContext* pRenderContext) if (isLast && pFrame->GetUpper() != GetUpper()) { assert(GetUpper()->Lower() == this + // empty section frames are created all the time... + || GetUpper()->Lower()->IsSctFrame() // tab frame/section frame may split multiple times || ( SwFlowFrame::CastFlowFrame(pFrame) && SwFlowFrame::CastFlowFrame(GetUpper()->Lower()) @@ -441,6 +443,8 @@ void SwFrame::PrepareCursor() if (isLast && pFrame->GetUpper() != GetUpper()) { assert(GetUpper()->Lower() == this + // empty section frames are created all the time... + || GetUpper()->Lower()->IsSctFrame() // tab frame/section frame may split multiple times || ( SwFlowFrame::CastFlowFrame(pFrame) && SwFlowFrame::CastFlowFrame(GetUpper()->Lower()) |