Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
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)
  • Loading branch information
brammool committed Apr 10, 2018
1 parent e80757c commit 8c87a2b
Show file tree
Hide file tree
Showing 5 changed files with 31 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/Makefile
Expand Up @@ -2127,6 +2127,7 @@ test_arglist \
test_autocmd \
test_autoload \
test_backspace_opt \
test_blockedit \
test_breakindent \
test_bufline \
test_bufwintabinfo \
Expand Down
10 changes: 7 additions & 3 deletions src/ops.c
Expand Up @@ -1093,7 +1093,7 @@ do_record(int c)

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 != '"'))
retval = FAIL;
else
Expand Down Expand Up @@ -2702,6 +2702,7 @@ op_insert(oparg_T *oap, long count1)
if (oap->block_mode)
{
struct block_def bd2;
int did_indent = FALSE;

/* If indent kicked in, the firstline might have changed
* but only do that, if the indent actually increased. */
Expand All @@ -2710,11 +2711,14 @@ op_insert(oparg_T *oap, long count1)
{
bd.textcol += 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
* to adjust the block for that. */
if (oap->start.lnum == curbuf->b_op_start_orig.lnum && !bd.is_MAX)
* to adjust the block for that. But only do it, if the difference
* 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
&& oap->start.col
Expand Down
1 change: 1 addition & 0 deletions src/testdir/Make_all.mak
Expand Up @@ -72,6 +72,7 @@ NEW_TESTS = test_arabic.res \
test_autocmd.res \
test_autoload.res \
test_backspace_opt.res \
test_blockedit.res \
test_breakindent.res \
test_bufwintabinfo.res \
test_cdo.res \
Expand Down
20 changes: 20 additions & 0 deletions src/testdir/test_blockedit.vim
@@ -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
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -762,6 +762,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
1682,
/**/
1681,
/**/
Expand Down

0 comments on commit 8c87a2b

Please sign in to comment.