mirror of
https://github.com/vim/vim.git
synced 2025-09-23 03:43:49 -04:00
patch 8.2.3198: cannot use 'formatlistpat' for breakindent
Problem: Cannot use 'formatlistpat' for breakindent. Solution: Use a negative list indent. (Maxim Kim, closes #8594)
This commit is contained in:
committed by
Bram Moolenaar
parent
d8e44476d8
commit
f674b358fc
@@ -1326,9 +1326,11 @@ A jump table for the options with a short description can be found at |Q_op|.
|
|||||||
continuation (positive).
|
continuation (positive).
|
||||||
sbr Display the 'showbreak' value before applying the
|
sbr Display the 'showbreak' value before applying the
|
||||||
additional indent.
|
additional indent.
|
||||||
list:{n} Adds an additional indent for lines that match a
|
list:{n} Adds an additional indent for lines that match a
|
||||||
numbered or bulleted list (using the
|
numbered or bulleted list (using the
|
||||||
'formatlistpat' setting).
|
'formatlistpat' setting).
|
||||||
|
list:-1 Uses the length of a match with 'formatlistpat'
|
||||||
|
for indentation.
|
||||||
The default value for min is 20, shift and list is 0.
|
The default value for min is 20, shift and list is 0.
|
||||||
|
|
||||||
*'browsedir'* *'bsdir'*
|
*'browsedir'* *'bsdir'*
|
||||||
|
18
src/indent.c
18
src/indent.c
@@ -941,15 +941,11 @@ get_breakindent_win(
|
|||||||
}
|
}
|
||||||
bri = prev_indent + wp->w_briopt_shift;
|
bri = prev_indent + wp->w_briopt_shift;
|
||||||
|
|
||||||
// indent minus the length of the showbreak string
|
|
||||||
if (wp->w_briopt_sbr)
|
|
||||||
bri -= vim_strsize(get_showbreak_value(wp));
|
|
||||||
|
|
||||||
// Add offset for number column, if 'n' is in 'cpoptions'
|
// Add offset for number column, if 'n' is in 'cpoptions'
|
||||||
bri += win_col_off2(wp);
|
bri += win_col_off2(wp);
|
||||||
|
|
||||||
// add additional indent for numbered lists
|
// add additional indent for numbered lists
|
||||||
if (wp->w_briopt_list > 0)
|
if (wp->w_briopt_list != 0)
|
||||||
{
|
{
|
||||||
regmatch_T regmatch;
|
regmatch_T regmatch;
|
||||||
|
|
||||||
@@ -958,11 +954,21 @@ get_breakindent_win(
|
|||||||
if (regmatch.regprog != NULL)
|
if (regmatch.regprog != NULL)
|
||||||
{
|
{
|
||||||
if (vim_regexec(®match, line, 0))
|
if (vim_regexec(®match, line, 0))
|
||||||
bri += wp->w_briopt_list;
|
{
|
||||||
|
if (wp->w_briopt_list > 0)
|
||||||
|
bri += wp->w_briopt_list;
|
||||||
|
else
|
||||||
|
bri = (*regmatch.endp - *regmatch.startp);
|
||||||
|
}
|
||||||
vim_regfree(regmatch.regprog);
|
vim_regfree(regmatch.regprog);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// indent minus the length of the showbreak string
|
||||||
|
if (wp->w_briopt_sbr)
|
||||||
|
bri -= vim_strsize(get_showbreak_value(wp));
|
||||||
|
|
||||||
|
|
||||||
// never indent past left window margin
|
// never indent past left window margin
|
||||||
if (bri < 0)
|
if (bri < 0)
|
||||||
bri = 0;
|
bri = 0;
|
||||||
|
@@ -759,6 +759,7 @@ func Test_breakindent20_list()
|
|||||||
\ ]
|
\ ]
|
||||||
let lines = s:screen_lines2(1, 9, 20)
|
let lines = s:screen_lines2(1, 9, 20)
|
||||||
call s:compare_lines(expect, lines)
|
call s:compare_lines(expect, lines)
|
||||||
|
|
||||||
" reset linebreak option
|
" reset linebreak option
|
||||||
" Note: it indents by one additional
|
" Note: it indents by one additional
|
||||||
" space, because of the leading space.
|
" space, because of the leading space.
|
||||||
@@ -775,7 +776,59 @@ func Test_breakindent20_list()
|
|||||||
let lines = s:screen_lines2(1, 6, 20)
|
let lines = s:screen_lines2(1, 6, 20)
|
||||||
call s:compare_lines(expect, lines)
|
call s:compare_lines(expect, lines)
|
||||||
|
|
||||||
call s:close_windows('set breakindent& briopt& linebreak& list& listchars&')
|
" check formatlistpat indent
|
||||||
|
setl briopt=min:5,list:-1
|
||||||
|
setl linebreak list&vim listchars&vim
|
||||||
|
let &l:flp = '^\s*\d\+\.\?[\]:)}\t ]\s*'
|
||||||
|
redraw!
|
||||||
|
let expect = [
|
||||||
|
\ " 1. Congress ",
|
||||||
|
\ " shall make no ",
|
||||||
|
\ " law ",
|
||||||
|
\ " 2.) Congress ",
|
||||||
|
\ " shall make no ",
|
||||||
|
\ " law ",
|
||||||
|
\ " 3.] Congress ",
|
||||||
|
\ " shall make no ",
|
||||||
|
\ " law ",
|
||||||
|
\ ]
|
||||||
|
let lines = s:screen_lines2(1, 9, 20)
|
||||||
|
call s:compare_lines(expect, lines)
|
||||||
|
" check formatlistpat indent with different list levels
|
||||||
|
let &l:flp = '^\s*\*\+\s\+'
|
||||||
|
redraw!
|
||||||
|
%delete _
|
||||||
|
call setline(1, ['* Congress shall make no law',
|
||||||
|
\ '*** Congress shall make no law',
|
||||||
|
\ '**** Congress shall make no law'])
|
||||||
|
norm! 1gg
|
||||||
|
let expect = [
|
||||||
|
\ "* Congress shall ",
|
||||||
|
\ " make no law ",
|
||||||
|
\ "*** Congress shall ",
|
||||||
|
\ " make no law ",
|
||||||
|
\ "**** Congress shall ",
|
||||||
|
\ " make no law ",
|
||||||
|
\ ]
|
||||||
|
let lines = s:screen_lines2(1, 6, 20)
|
||||||
|
call s:compare_lines(expect, lines)
|
||||||
|
|
||||||
|
" check formatlistpat indent with different list level
|
||||||
|
" showbreak and sbr
|
||||||
|
setl briopt=min:5,sbr,list:-1,shift:2
|
||||||
|
setl showbreak=>
|
||||||
|
redraw!
|
||||||
|
let expect = [
|
||||||
|
\ "* Congress shall ",
|
||||||
|
\ "> make no law ",
|
||||||
|
\ "*** Congress shall ",
|
||||||
|
\ "> make no law ",
|
||||||
|
\ "**** Congress shall ",
|
||||||
|
\ "> make no law ",
|
||||||
|
\ ]
|
||||||
|
let lines = s:screen_lines2(1, 6, 20)
|
||||||
|
call s:compare_lines(expect, lines)
|
||||||
|
call s:close_windows('set breakindent& briopt& linebreak& list& listchars& showbreak&')
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
" vim: shiftwidth=2 sts=2 expandtab
|
" vim: shiftwidth=2 sts=2 expandtab
|
||||||
|
@@ -755,6 +755,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 */
|
||||||
|
/**/
|
||||||
|
3198,
|
||||||
/**/
|
/**/
|
||||||
3197,
|
3197,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user