1
0
forked from aniani/vim

patch 8.2.4955: text property in wrong position after auto-indent

Problem:    Text property in wrong position after auto-indent.
Solution:   Adjust text property columns. (closes #10422, closes #7719)
This commit is contained in:
LemonBoy
2022-05-14 18:48:05 +01:00
committed by Bram Moolenaar
parent 698cb4c8fa
commit 788c06a249
3 changed files with 24 additions and 3 deletions

View File

@@ -1372,7 +1372,8 @@ open_line(
char_u *next_line = NULL; // copy of the next line char_u *next_line = NULL; // copy of the next line
char_u *p_extra = NULL; // what goes to next line char_u *p_extra = NULL; // what goes to next line
int less_cols = 0; // less columns for mark in new line int less_cols = 0; // less columns for mark in new line
int less_cols_off = 0; // columns to skip for mark adjust int less_cols_off = 0; // columns to skip for mark and
// textprop adjustment
pos_T old_cursor; // old cursor position pos_T old_cursor; // old cursor position
int newcol = 0; // new cursor column int newcol = 0; // new cursor column
int newindent = 0; // auto-indent of the new line int newindent = 0; // auto-indent of the new line
@@ -2151,9 +2152,9 @@ open_line(
did_append = TRUE; did_append = TRUE;
#ifdef FEAT_PROP_POPUP #ifdef FEAT_PROP_POPUP
if ((State & MODE_INSERT) && (State & VREPLACE_FLAG) == 0) if ((State & MODE_INSERT) && (State & VREPLACE_FLAG) == 0)
// properties after the split move to the next line // Properties after the split move to the next line.
adjust_props_for_split(curwin->w_cursor.lnum, curwin->w_cursor.lnum, adjust_props_for_split(curwin->w_cursor.lnum, curwin->w_cursor.lnum,
curwin->w_cursor.col + 1, 0); curwin->w_cursor.col + 1, 0);
#endif #endif
} }
else else
@@ -2248,6 +2249,12 @@ open_line(
mark_col_adjust(curwin->w_cursor.lnum, mark_col_adjust(curwin->w_cursor.lnum,
curwin->w_cursor.col + less_cols_off, curwin->w_cursor.col + less_cols_off,
1L, (long)-less_cols, 0); 1L, (long)-less_cols, 0);
#ifdef FEAT_PROP_POPUP
// Keep into account the deleted blanks on the new line.
if (curbuf->b_has_textprop && less_cols_off != 0)
adjust_prop_columns(curwin->w_cursor.lnum + 1, 0,
-less_cols_off, 0);
#endif
} }
else else
changed_bytes(curwin->w_cursor.lnum, curwin->w_cursor.col); changed_bytes(curwin->w_cursor.lnum, curwin->w_cursor.col);

View File

@@ -635,6 +635,18 @@ func Test_prop_open_line()
call assert_equal(expected, prop_list(2)) call assert_equal(expected, prop_list(2))
call DeletePropTypes() call DeletePropTypes()
" split at the space character with 'ai' active, the leading space is removed
" in the second line and the prop is shifted accordingly.
let expected = SetupOneLine() " 'xonex xtwoxx'
set ai
exe "normal 6|i\<CR>\<Esc>"
call assert_equal('xonex', getline(1))
call assert_equal('xtwoxx', getline(2))
let expected[1].col -= 6
call assert_equal(expected, prop_list(1) + prop_list(2))
set ai&
call DeletePropTypes()
bwipe! bwipe!
set bs& set bs&
endfunc endfunc

View File

@@ -746,6 +746,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 */
/**/
4955,
/**/ /**/
4954, 4954,
/**/ /**/