Bug 121062 - The time to cut a large amount of text including footnotes has increased from 15 to 24 seconds
Summary: The time to cut a large amount of text including footnotes has increased from...
Status: RESOLVED NOTABUG
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.2.0.0.alpha1+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, perf, regression
Depends on:
Blocks: Cut-Copy
  Show dependency tree / graph
 
Reported: 2018-10-30 15:05 UTC by Telesto
Modified: 2022-09-25 07:36 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2018-10-30 15:05:19 UTC 
Description:
The to cut a large amount of text including footnotes has increased from 15 tot 24 seconds

Steps to Reproduce:
1. Open the attachment 145863 [details]
2. Wait until the background processing is done
3. Press CTRL+A (wait until the background processing is done)
4. CTR+X

Actual Results:
24 seconds

Expected Results:
15 seconds 


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 6.2.0.0.alpha1+
Build ID: 2e184378da6058af14acf208d6469fadf44d04ed
CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2018-10-30_03:29:39
Locale: nl-NL (nl_NL); Calc: CL

but not in
Version: 6.1.0.0.beta2+
Build ID: 22c451df33b733440f24c1feb6380d31240d55e6
CPU threads: 4; OS: Windows 6.3; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL
Comment 1 Durgapriyanka 2018-11-29 17:22:57 UTC 
I can not confirm this with my version, because every time I press CTRL+X, after 7 to 10 seconds, it says not responding. This happens every time I try this. 

Version: 6.0.6.2
Build ID: 0c292870b25a325b5ed35f6b45599d2ea4458e77
CPU threads: 2; OS: Windows 6.1; UI render: default; 
Locale: en-US (en_US); Calc: group
Comment 2 Dieter 2018-12-05 21:46:24 UTC 
I confirm it

LO 6.2.0.0Beta 1: 40 sec.
LO 5.4.7.2: 25 sec.
Comment 3 Buovjaga 2019-01-25 16:09:29 UTC 
Bibisected with win32-6.2 to https://gerrit.libreoffice.org/plugins/gitiles/core/+/847848d1401f23d51501e4b3396967a4bc193913%5E!/

make SwContentNode handle legacy ModifyNotification() again (tdf#117749)

Double-checked by checking out the commit and previous one.

Adding Cc: to Bjoern Michaelsen

Seems to be the same case with Linux.

Arch Linux 64-bit
Version: 6.3.0.0.alpha0+
Build ID: bb30e9e591d5f9f913b3cd8fbaa3c5e412b509bd
CPU threads: 8; OS: Linux 4.20; UI render: default; VCL: gtk3; 
Locale: fi-FI (fi_FI.UTF-8); UI-Language: en-US
Calc: threaded
Built on 23 January 2019
Comment 4 Dieter 2022-08-16 07:09:14 UTC 
Still present in

Version: 7.3.5.2 (x64) / LibreOffice Community
Build ID: 184fe81b8c8c30d8b5082578aee2fed2ea847c01
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: en-GB
Calc: CL

30 seconds, so little improvement but still not sufficient.
Comment 5 Björn Michaelsen 2022-09-25 01:37:36 UTC 
(In reply to Buovjaga from comment #3)
> Bibisected with win32-6.2 to
> https://gerrit.libreoffice.org/plugins/gitiles/core/+/
> 847848d1401f23d51501e4b3396967a4bc193913%5E!/

That fixed a crash -- it should be not be too controversial that is better to have it take a bit longer in this corner case than having the product crash.
Comment 6 Buovjaga 2022-09-25 07:36:21 UTC 
Ok, let's close.