0
0
mirror of https://github.com/vim/vim.git synced 2025-10-11 06:34:16 -04:00

patch 8.0.1682: auto indenting breaks inserting a block

Problem:    Auto indenting breaks inserting a block.
Solution:   Do not check for cursor movement if indent was changed. (Christian
            Brabandt, closes #2778)
This commit is contained in:
Bram Moolenaar
2018-04-10 13:15:47 +02:00
parent e80757c154
commit 8c87a2b1fe
5 changed files with 31 additions and 3 deletions

View File

@@ -2127,6 +2127,7 @@ test_arglist \
test_autocmd \ test_autocmd \
test_autoload \ test_autoload \
test_backspace_opt \ test_backspace_opt \
test_blockedit \
test_breakindent \ test_breakindent \
test_bufline \ test_bufline \
test_bufwintabinfo \ test_bufwintabinfo \

View File

@@ -1093,7 +1093,7 @@ do_record(int c)
if (Recording == FALSE) /* start recording */ if (Recording == FALSE) /* start recording */
{ {
/* registers 0-9, a-z and " are allowed */ /* registers 0-9, a-z and " are allowed */
if (c < 0 || (!ASCII_ISALNUM(c) && c != '"')) if (c < 0 || (!ASCII_ISALNUM(c) && c != '"'))
retval = FAIL; retval = FAIL;
else else
@@ -2702,6 +2702,7 @@ op_insert(oparg_T *oap, long count1)
if (oap->block_mode) if (oap->block_mode)
{ {
struct block_def bd2; struct block_def bd2;
int did_indent = FALSE;
/* If indent kicked in, the firstline might have changed /* If indent kicked in, the firstline might have changed
* but only do that, if the indent actually increased. */ * but only do that, if the indent actually increased. */
@@ -2710,11 +2711,14 @@ op_insert(oparg_T *oap, long count1)
{ {
bd.textcol += ind_post - ind_pre; bd.textcol += ind_post - ind_pre;
bd.start_vcol += ind_post - ind_pre; bd.start_vcol += ind_post - ind_pre;
did_indent = TRUE;
} }
/* The user may have moved the cursor before inserting something, try /* The user may have moved the cursor before inserting something, try
* to adjust the block for that. */ * to adjust the block for that. But only do it, if the difference
if (oap->start.lnum == curbuf->b_op_start_orig.lnum && !bd.is_MAX) * does not come from indent kicking in. */
if (oap->start.lnum == curbuf->b_op_start_orig.lnum
&& !bd.is_MAX && !did_indent)
{ {
if (oap->op_type == OP_INSERT if (oap->op_type == OP_INSERT
&& oap->start.col && oap->start.col

View File

@@ -72,6 +72,7 @@ NEW_TESTS = test_arabic.res \
test_autocmd.res \ test_autocmd.res \
test_autoload.res \ test_autoload.res \
test_backspace_opt.res \ test_backspace_opt.res \
test_blockedit.res \
test_breakindent.res \ test_breakindent.res \
test_bufwintabinfo.res \ test_bufwintabinfo.res \
test_cdo.res \ test_cdo.res \

View File

@@ -0,0 +1,20 @@
" Test for block inserting
"
" TODO: rewrite test39.in into this new style test
func Test_blockinsert_indent()
new
filetype plugin indent on
setlocal sw=2 et ft=vim
call setline(1, ['let a=[', ' ''eins'',', ' ''zwei'',', ' ''drei'']'])
call cursor(2, 3)
exe "norm! \<c-v>2jI\\ \<esc>"
call assert_equal(['let a=[', ' \ ''eins'',', ' \ ''zwei'',', ' \ ''drei'']'],
\ getline(1,'$'))
" reset to sane state
filetype off
bwipe!
endfunc
" vim: shiftwidth=2 sts=2 expandtab

View File

@@ -762,6 +762,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
1682,
/**/ /**/
1681, 1681,
/**/ /**/