mirror of
https://github.com/vim/vim.git
synced 2025-09-24 03:44:06 -04:00
patch 8.2.3400: ":z!" is not supported
Problem: ":z!" is not supported. Solution: Make ":z!" work and add tests. (Dominique Pellé, closes #8836) Use display height instead of current window height.
This commit is contained in:
committed by
Bram Moolenaar
parent
deba5eb195
commit
7f2dd1e90c
@@ -170,8 +170,13 @@ g8 Print the hex values of the bytes used in the
|
|||||||
If the mark is "=", a line of dashes is printed
|
If the mark is "=", a line of dashes is printed
|
||||||
around the current line.
|
around the current line.
|
||||||
|
|
||||||
:[range]z#[+-^.=][count] *:z#*
|
*:z!
|
||||||
Like ":z", but number the lines.
|
:[range]z![+-^.=][count]
|
||||||
|
Like ":z:", but when [count] is not specified, it
|
||||||
|
defaults to the Vim window height minus one.
|
||||||
|
|
||||||
|
:[range]z[!]#[+-^.=][count] *:z#*
|
||||||
|
Like ":z" or ":z!", but number the lines.
|
||||||
|
|
||||||
*:=*
|
*:=*
|
||||||
:= [flags] Print the last line number.
|
:= [flags] Print the last line number.
|
||||||
|
@@ -3445,7 +3445,7 @@ ex_z(exarg_T *eap)
|
|||||||
// Vi compatible: ":z!" uses display height, without a count uses
|
// Vi compatible: ":z!" uses display height, without a count uses
|
||||||
// 'scroll'
|
// 'scroll'
|
||||||
if (eap->forceit)
|
if (eap->forceit)
|
||||||
bigness = curwin->w_height;
|
bigness = Rows - 1;
|
||||||
else if (!ONE_WINDOW)
|
else if (!ONE_WINDOW)
|
||||||
bigness = curwin->w_height - 3;
|
bigness = curwin->w_height - 3;
|
||||||
else
|
else
|
||||||
|
@@ -1824,7 +1824,7 @@ EXCMD(CMD_yank, "yank", ex_operators,
|
|||||||
EX_RANGE|EX_WHOLEFOLD|EX_REGSTR|EX_COUNT|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
|
EX_RANGE|EX_WHOLEFOLD|EX_REGSTR|EX_COUNT|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
|
||||||
ADDR_LINES),
|
ADDR_LINES),
|
||||||
EXCMD(CMD_z, "z", ex_z,
|
EXCMD(CMD_z, "z", ex_z,
|
||||||
EX_RANGE|EX_WHOLEFOLD|EX_EXTRA|EX_FLAGS|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
|
EX_RANGE|EX_WHOLEFOLD|EX_BANG|EX_EXTRA|EX_FLAGS|EX_TRLBAR|EX_CMDWIN|EX_LOCK_OK,
|
||||||
ADDR_LINES),
|
ADDR_LINES),
|
||||||
|
|
||||||
// commands that don't start with a letter
|
// commands that don't start with a letter
|
||||||
|
@@ -16,8 +16,9 @@ func Test_z()
|
|||||||
call assert_equal(23, line('.'))
|
call assert_equal(23, line('.'))
|
||||||
|
|
||||||
let a = execute('20z+3')
|
let a = execute('20z+3')
|
||||||
" FIXME: I would expect the same result as '20z3' but it
|
" FIXME: I would expect the same result as '20z3' since 'help z'
|
||||||
" gives "\n21\n22\n23" instead. Bug in Vim or in ":help :z"?
|
" says: Specifying no mark at all is the same as "+".
|
||||||
|
" However it " gives "\n21\n22\n23" instead. Bug in Vim or in ":help :z"?
|
||||||
"call assert_equal("\n20\n21\n22", a)
|
"call assert_equal("\n20\n21\n22", a)
|
||||||
"call assert_equal(22, line('.'))
|
"call assert_equal(22, line('.'))
|
||||||
|
|
||||||
@@ -55,19 +56,48 @@ func Test_z()
|
|||||||
call assert_equal(100, line('.'))
|
call assert_equal(100, line('.'))
|
||||||
|
|
||||||
let a = execute('20z-1000')
|
let a = execute('20z-1000')
|
||||||
call assert_match("^\n1\n2\n.*\n19\n20$", a)
|
|
||||||
call assert_equal(20, line('.'))
|
call assert_equal(20, line('.'))
|
||||||
|
|
||||||
let a = execute('20z=1000')
|
let a = execute('20z=1000')
|
||||||
call assert_match("^\n1\n.*\n-\\+\n20\n-\\\+\n.*\n100$", a)
|
call assert_match("^\n1\n.*\n-\\+\n20\n-\\\+\n.*\n100$", a)
|
||||||
call assert_equal(20, line('.'))
|
call assert_equal(20, line('.'))
|
||||||
|
|
||||||
|
" Tests with multiple windows.
|
||||||
|
5split
|
||||||
|
call setline(1, range(1, 100))
|
||||||
|
" Without a count, the number line is window height - 3.
|
||||||
|
let a = execute('20z')
|
||||||
|
call assert_equal("\n20\n21", a)
|
||||||
|
call assert_equal(21, line('.'))
|
||||||
|
" If window height - 3 is less than 1, it should be clamped to 1.
|
||||||
|
resize 2
|
||||||
|
let a = execute('20z')
|
||||||
|
call assert_equal("\n20", a)
|
||||||
|
call assert_equal(20, line('.'))
|
||||||
|
|
||||||
call assert_fails('20z=a', 'E144:')
|
call assert_fails('20z=a', 'E144:')
|
||||||
|
|
||||||
set window& scroll&
|
set window& scroll&
|
||||||
bw!
|
bw!
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
|
" :z! is the same as :z but count uses the Vim window height when not specified.
|
||||||
|
func Test_z_bang()
|
||||||
|
4split
|
||||||
|
call setline(1, range(1, 20))
|
||||||
|
|
||||||
|
let a = execute('10z!')
|
||||||
|
call assert_equal("\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20", a)
|
||||||
|
|
||||||
|
let a = execute('10z!#')
|
||||||
|
call assert_equal("\n 10 10\n 11 11\n 12 12\n 13 13\n 14 14\n 15 15\n 16 16\n 17 17\n 18 18\n 19 19\n 20 20", a)
|
||||||
|
|
||||||
|
let a = execute('10z!3')
|
||||||
|
call assert_equal("\n10\n11\n12", a)
|
||||||
|
|
||||||
|
%bwipe!
|
||||||
|
endfunc
|
||||||
|
|
||||||
func Test_z_overflow()
|
func Test_z_overflow()
|
||||||
" This used to access invalid memory as a result of an integer overflow
|
" This used to access invalid memory as a result of an integer overflow
|
||||||
" and freeze vim.
|
" and freeze vim.
|
||||||
|
@@ -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 */
|
||||||
|
/**/
|
||||||
|
3400,
|
||||||
/**/
|
/**/
|
||||||
3399,
|
3399,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user