summaryrefslogtreecommitdiffstats
path: root/basic
diff options
context:
space:
mode:
authorMichael Stahl <michael.stahl@allotropia.de>2021-06-25 10:25:59 +0200
committerThorsten Behrens <thorsten.behrens@allotropia.de>2021-06-28 09:40:39 +0200
commit7e3efa231c6e9b7944c63f3ac67cf12dc396aabf (patch)
tree2b08dc1102ff611f611af3514af031ba95985f76 /basic
parentUpdate git submodules (diff)
downloadcore-7e3efa231c6e9b7944c63f3ac67cf12dc396aabf.tar.gz
core-7e3efa231c6e9b7944c63f3ac67cf12dc396aabf.zip
sw: layout: fix table split loop caused by RemoveFollowFlowLinePending
In no_view.odt, the table 1340 does RemoveFollowFlowLine(), then splits again, and by the time Split() returns, its flag m_bRemoveFollowFlowLinePending is set and so in the next SwTabFrame::MakeAll() it will do the same thing again, until eventually it hits layact.cxx:544: LoopControl_1 in SwLayAction::InternalAction The flag is set during lcl_RecalcSplitLine(), in SwContentFrame::Cut(), which operates under the assumption that there are no other cells in the follow flow line that may contain content frames. This only happens if "Verdana" and "Arial" fonts are available and not substituted with "DejaVu Sans" etc. (regression from c4af8b0f106f31b6e79a145bc4ab83bb6a55c808) Unfortunately this uncovers another pagination bug in testTdf88496 which was hidden by this flag being set from text frames that shouldn't exist anyway. Change-Id: I3cc82c13bfc06960b827aefc64f5b5b12520955a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117851 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@allotropia.de> (cherry picked from commit a0cd270a99f43049fc099ddd463e57aa243d5b0e) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117859 Reviewed-by: Thorsten Behrens <thorsten.behrens@allotropia.de>
Diffstat (limited to 'basic')
0 files changed, 0 insertions, 0 deletions