Bug 133441

Summary: FILEOPEN: DOC: Hang at import time of table in table
Product: LibreOffice Reporter: Xisco Faulí <xiscofauli>
Component: WriterAssignee: Justin L <jluth>
Status: VERIFIED FIXED    
Severity: normal CC: jluth, xiscofauli
Priority: medium Keywords: bibisected, bisected, perf, regression
Version: 7.0.0.0.alpha0+   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=105478
Whiteboard:
Crash report or crash signature: Regression By:

Description Xisco Faulí 2020-05-27 13:23:44 UTC 
Steps to reproduce:
1. Open attachment 52300 [details] from bug 41748

-> LibreOffice hangs forever ( killed after 11 minutes)

Reproduced in

Version: 7.0.0.0.alpha1+
Build ID: 82894d85147840f1f587e9530b12f0058f2ef2c3
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded


[Bug found by office-interoperability-tools]
Comment 1 Xisco Faulí 2020-05-27 13:27:49 UTC 
Regression introduced by:

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

author	Justin Luth <justin.luth@collabora.com>	2020-05-04 13:16:32 +0300
committer	Miklos Vajna <vmiklos@collabora.com>	2020-05-11 10:14:19 +0200
commit aa5dc0b48cd4db6883c9b52c68b16170c9c81d1f (patch)
tree 7a1bb6161ef3bad60db5c1093a1a5c7d2e9dcec8
parent 356edfa963bd1d1c8e081a012e6a2ccbe7bf53d3 (diff)
tdf#105478 sw layout: treat minHeight as "do not split row"

Bisected with: bibisect-linux64-6.5

Adding Cc: to Justin Luth
Comment 2 Justin L 2020-05-27 18:27:18 UTC 
Created attachment 161353 [details]
A table in table with a minimum row height.doc

Comment 0's example document is a table-in-table-in-table nightmare. However, my simple example document shows that I can't ignore minimum row height simply because it is table-in-table.  Too bad.

LO could open a version that was first round-tripped by Word 2003, but that isn't really important. But it suggests that this just happens to trigger a layout state that LO can't handle. I'm not sure what to do or try next.
Comment 3 Justin L 2020-05-30 11:30:17 UTC 
A revert patch is queued up at https://gerrit.libreoffice.org/c/core/+/95180, which will be my fallback if this cannot be properly fixed.
Comment 4 Justin L 2020-06-03 13:50:09 UTC 
patch has been reverted.
Comment 5 Xisco Faulí 2020-06-09 10:43:05 UTC 
Verified in

Version: 7.1.0.0.alpha0+
Build ID: e0f058bcd5e185b89e4b7c680a7901bc7eb2a790
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

@Justin L, thanks for fixing this issue!