mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-09-06 03:18:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

vim-patch:8.2.4792: indent operator creates an undo entry for every line

Browse Source 
Problem:    Indent operator creates an undo entry for every line.
Solution:   Create one undo entry for all lines. (closes vim/vim#10227)
e468698994
This commit is contained in:
zeertzjq
2022-04-20 08:16:41 +08:00
parent c72857d396
commit f8b832b171
2 changed files with 53 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

57
src/nvim/ops.c
View File

@@ -649,38 +649,41 @@ void op_reindent(oparg_T *oap, Indenter how)
return; return;
} }
for (i = oap->line_count - 1; i >= 0 && !got_int; i--) { // Save for undo. Do this once for all lines, much faster than doing this
/* it's a slow thing to do, so give feedback so there's no worry that // for each line separately, especially when undoing.
* the computer's just hung. */ if (u_savecommon(curbuf, start_lnum - 1, start_lnum + oap->line_count,
start_lnum + oap->line_count, false) == OK) {
for (i = oap->line_count - 1; i >= 0 && !got_int; i--) {
// it's a slow thing to do, so give feedback so there's no worry
// that the computer's just hung.
if (i > 1 if (i > 1
&& (i % 50 == 0 || i == oap->line_count - 1) && (i % 50 == 0 || i == oap->line_count - 1)
&& oap->line_count > p_report) { && oap->line_count > p_report) {
smsg(_("%" PRId64 " lines to indent... "), (int64_t)i); smsg(_("%" PRId64 " lines to indent... "), (int64_t)i);
}
/*
* Be vi-compatible: For lisp indenting the first line is not
* indented, unless there is only one line.
*/
if (i != oap->line_count - 1 || oap->line_count == 1
|| how != get_lisp_indent) {
l = skipwhite(get_cursor_line_ptr());
if (*l == NUL) { // empty or blank line
amount = 0;
} else {
amount = how(); // get the indent for this line
} }
if (amount >= 0 && set_indent(amount, SIN_UNDO)) {
// did change the indent, call changed_lines() later // Be vi-compatible: For lisp indenting the first line is not
if (first_changed == 0) { // indented, unless there is only one line.
first_changed = curwin->w_cursor.lnum; if (i != oap->line_count - 1 || oap->line_count == 1
|| how != get_lisp_indent) {
l = skipwhite(get_cursor_line_ptr());
if (*l == NUL) { // empty or blank line
amount = 0;
} else {
amount = how(); // get the indent for this line
}
if (amount >= 0 && set_indent(amount, 0)) {
// did change the indent, call changed_lines() later
if (first_changed == 0) {
first_changed = curwin->w_cursor.lnum;
}
last_changed = curwin->w_cursor.lnum;
} }
last_changed = curwin->w_cursor.lnum;
} }
curwin->w_cursor.lnum++;
curwin->w_cursor.col = 0; // make sure it's valid
} }
++curwin->w_cursor.lnum;
curwin->w_cursor.col = 0; // make sure it's valid
} }
// put cursor on first non-blank of indented line // put cursor on first non-blank of indented line

23
src/nvim/testdir/test_indent.vim
View File

@@ -59,6 +59,29 @@ func Test_reindent()
close! close!
endfunc endfunc
" Test indent operator creating one undo entry
func Test_indent_operator_undo()
enew
call setline(1, range(12)->map('"\t" .. v:val'))
func FoldExpr()
let g:foldcount += 1
return '='
endfunc
set foldmethod=expr foldexpr=FoldExpr()
let g:foldcount = 0
redraw
call assert_equal(12, g:foldcount)
normal gg=G
call assert_equal(24, g:foldcount)
undo
call assert_equal(38, g:foldcount)
bwipe!
set foldmethod& foldexpr=
delfunc FoldExpr
unlet g:foldcount
endfunc
" Test for shifting a line with a preprocessor directive ('#') " Test for shifting a line with a preprocessor directive ('#')
func Test_preproc_indent() func Test_preproc_indent()
new new