anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-12-23 13:31:49 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Implement undo coalescing

Browse Source 
if the undo element we want to add only changes stuff that was already
modified by the previous one on undo stack (in the same group), then
skip it. There is nothing to gain in adding it to the stack.

The typical use case is when doing a search and replace in a large
document that does many replacements in each paragraph. In this case,
the same paragraph would be stored repeatedly.

Fixes bug #12564.
This commit is contained in:
Jean-Marc Lasgouttes 2022-07-14 01:02:28 +02:00
parent 4c1db7cad8
commit f5bbadbad9
1 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/Undo.cpp
View File

@ -312,12 +312,29 @@ void Undo::Private::doRecordUndo(UndoKind kind,
if (first_pit > last_pit)
swap(first_pit, last_pit);
pit_type const from = first_pit;
pit_type const end = cell.lastpit() - last_pit;
/* Undo coalescing: if the undo element we want to add only
* changes stuff that was already modified by the previous one on
* undo stack (in the same group), then skip it. There is nothing
* to gain in adding it to the stack. The code below works for
* both texted and mathed.
*/
if (!stack.empty()
&& stack.top().group_id == group_id_
&& !stack.top().bparams
&& samePar(stack.top().cell, cell)
//&& stack.top().kind == kind // needed?
&& stack.top().from <= from
&& stack.top().end >= end) {
LYXERR(Debug::UNDO, "Undo coalescing: skip entry");
return;
}
// Undo::ATOMIC are always recorded (no overlapping there).
// As nobody wants all removed character appear one by one when undoing,
// we want combine 'similar' non-ATOMIC undo recordings to one.
pit_type from = first_pit;
pit_type end = cell.lastpit() - last_pit;
if (!undo_finished_
&& kind != ATOMIC_UNDO
&& !stack.empty()