mirror of
https://github.com/neovim/neovim.git
synced 2024-12-24 05:05:00 -07:00
fix(pager): handle consecutive newlines properly (#27913)
This commit is contained in:
zeertzjq
GitHub
parent
5de0482d1a
commit
e3bd04f2af
@ -8085,10 +8085,12 @@ void ex_echo(exarg_T *eap)
|
|||||||
// Call msg_start() after eval1(), evaluating the expression
|
// Call msg_start() after eval1(), evaluating the expression
|
||||||
// may cause a message to appear.
|
// may cause a message to appear.
|
||||||
if (eap->cmdidx == CMD_echo) {
|
if (eap->cmdidx == CMD_echo) {
|
||||||
// Mark the saved text as finishing the line, so that what
|
if (!msg_didout) {
|
||||||
// follows is displayed on a new line when scrolling back
|
// Mark the saved text as finishing the line, so that what
|
||||||
// at the more prompt.
|
// follows is displayed on a new line when scrolling back
|
||||||
msg_sb_eol();
|
// at the more prompt.
|
||||||
|
msg_sb_eol();
|
||||||
|
}
|
||||||
msg_start();
|
msg_start();
|
||||||
}
|
}
|
||||||
} else if (eap->cmdidx == CMD_echo) {
|
} else if (eap->cmdidx == CMD_echo) {
|
||||||
|
|||||||
@ -1191,7 +1191,7 @@ void wait_return(int redraw)
|
|||||||
check_timestamps(false);
|
check_timestamps(false);
|
||||||
}
|
}
|
||||||
|
|
||||||
hit_return_msg();
|
hit_return_msg(true);
|
||||||
|
|
||||||
do {
|
do {
|
||||||
// Remember "got_int", if it is set vgetc() probably returns a
|
// Remember "got_int", if it is set vgetc() probably returns a
|
||||||
@ -1240,7 +1240,7 @@ void wait_return(int redraw)
|
|||||||
got_int = false;
|
got_int = false;
|
||||||
} else if (c != K_IGNORE) {
|
} else if (c != K_IGNORE) {
|
||||||
c = K_IGNORE;
|
c = K_IGNORE;
|
||||||
hit_return_msg();
|
hit_return_msg(false);
|
||||||
}
|
}
|
||||||
} else if (msg_scrolled > Rows - 2
|
} else if (msg_scrolled > Rows - 2
|
||||||
&& (c == 'j' || c == 'd' || c == 'f'
|
&& (c == 'j' || c == 'd' || c == 'f'
|
||||||
@ -1313,14 +1313,19 @@ void wait_return(int redraw)
|
|||||||
}
|
}
|
||||||
|
|
||||||
/// Write the hit-return prompt.
|
/// Write the hit-return prompt.
|
||||||
static void hit_return_msg(void)
|
///
|
||||||
|
/// @param newline_sb if starting a new line, add it to the scrollback.
|
||||||
|
static void hit_return_msg(bool newline_sb)
|
||||||
{
|
{
|
||||||
int save_p_more = p_more;
|
int save_p_more = p_more;
|
||||||
|
|
||||||
p_more = false; // don't want to see this message when scrolling back
|
if (!newline_sb) {
|
||||||
|
p_more = false;
|
||||||
|
}
|
||||||
if (msg_didout) { // start on a new line
|
if (msg_didout) { // start on a new line
|
||||||
msg_putchar('\n');
|
msg_putchar('\n');
|
||||||
}
|
}
|
||||||
|
p_more = false; // don't want to see this message when scrolling back
|
||||||
msg_ext_set_kind("return_prompt");
|
msg_ext_set_kind("return_prompt");
|
||||||
if (got_int) {
|
if (got_int) {
|
||||||
msg_puts(_("Interrupt: "));
|
msg_puts(_("Interrupt: "));
|
||||||
@ -2968,7 +2973,7 @@ void repeat_message(void)
|
|||||||
msg_col = 0;
|
msg_col = 0;
|
||||||
msg_clr_eos();
|
msg_clr_eos();
|
||||||
}
|
}
|
||||||
hit_return_msg();
|
hit_return_msg(false);
|
||||||
msg_row = Rows - 1;
|
msg_row = Rows - 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -10,6 +10,7 @@ local api = helpers.api
|
|||||||
local async_meths = helpers.async_meths
|
local async_meths = helpers.async_meths
|
||||||
local test_build_dir = helpers.paths.test_build_dir
|
local test_build_dir = helpers.paths.test_build_dir
|
||||||
local nvim_prog = helpers.nvim_prog
|
local nvim_prog = helpers.nvim_prog
|
||||||
|
local testprg = helpers.testprg
|
||||||
local exec = helpers.exec
|
local exec = helpers.exec
|
||||||
local exec_capture = helpers.exec_capture
|
local exec_capture = helpers.exec_capture
|
||||||
local exc_exec = helpers.exc_exec
|
local exc_exec = helpers.exc_exec
|
||||||
@ -2498,6 +2499,178 @@ aliquip ex ea commodo consequat.]]
|
|||||||
]],
|
]],
|
||||||
}
|
}
|
||||||
end)
|
end)
|
||||||
|
|
||||||
|
it('g< shows blank line from :echo properly', function()
|
||||||
|
screen:try_resize(60, 8)
|
||||||
|
feed([[:echo 1 | echo "\n" | echo 2<CR>]])
|
||||||
|
screen:expect([[
|
||||||
|
|
|
||||||
|
{1:~ }|*2
|
||||||
|
{12: }|
|
||||||
|
1 |
|
||||||
|
|
|
||||||
|
2 |
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
|
||||||
|
feed('<CR>')
|
||||||
|
screen:expect([[
|
||||||
|
^ |
|
||||||
|
{1:~ }|*6
|
||||||
|
|
|
||||||
|
]])
|
||||||
|
|
||||||
|
feed('g<lt>')
|
||||||
|
screen:expect([[
|
||||||
|
|
|
||||||
|
{1:~ }|
|
||||||
|
{12: }|
|
||||||
|
:echo 1 | echo "\n" | echo 2 |
|
||||||
|
1 |
|
||||||
|
|
|
||||||
|
2 |
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
|
||||||
|
feed('<CR>')
|
||||||
|
screen:expect([[
|
||||||
|
^ |
|
||||||
|
{1:~ }|*6
|
||||||
|
|
|
||||||
|
]])
|
||||||
|
end)
|
||||||
|
|
||||||
|
it('scrolling works properly when :echo output ends with newline', function()
|
||||||
|
screen:try_resize(60, 6)
|
||||||
|
feed([[:echo range(100)->join("\n") .. "\n"<CR>]])
|
||||||
|
screen:expect([[
|
||||||
|
0 |
|
||||||
|
1 |
|
||||||
|
2 |
|
||||||
|
3 |
|
||||||
|
4 |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('G')
|
||||||
|
screen:expect([[
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
99 |
|
||||||
|
|
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
for _ = 1, 3 do
|
||||||
|
feed('k')
|
||||||
|
screen:expect([[
|
||||||
|
95 |
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
99 |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('k')
|
||||||
|
screen:expect([[
|
||||||
|
94 |
|
||||||
|
95 |
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
95 |
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
99 |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
99 |
|
||||||
|
|
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
96 |
|
||||||
|
97 |
|
||||||
|
98 |
|
||||||
|
99 |
|
||||||
|
|
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
end
|
||||||
|
end)
|
||||||
|
|
||||||
|
it('scrolling works properly when :!cmd output ends with newline #27902', function()
|
||||||
|
screen:try_resize(60, 6)
|
||||||
|
api.nvim_set_option_value('shell', testprg('shell-test'), {})
|
||||||
|
api.nvim_set_option_value('shellcmdflag', 'REP 100', {})
|
||||||
|
api.nvim_set_option_value('shellxquote', '', {}) -- win: avoid extra quotes
|
||||||
|
feed([[:!foo<CR>]])
|
||||||
|
screen:expect([[
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
99: foo |
|
||||||
|
|
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
for _ = 1, 3 do
|
||||||
|
feed('k')
|
||||||
|
screen:expect([[
|
||||||
|
95: foo |
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
99: foo |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('k')
|
||||||
|
screen:expect([[
|
||||||
|
94: foo |
|
||||||
|
95: foo |
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
95: foo |
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
99: foo |
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
99: foo |
|
||||||
|
|
|
||||||
|
{4:-- More --}^ |
|
||||||
|
]])
|
||||||
|
feed('j')
|
||||||
|
screen:expect([[
|
||||||
|
96: foo |
|
||||||
|
97: foo |
|
||||||
|
98: foo |
|
||||||
|
99: foo |
|
||||||
|
|
|
||||||
|
{4:Press ENTER or type command to continue}^ |
|
||||||
|
]])
|
||||||
|
end
|
||||||
|
end)
|
||||||
end)
|
end)
|
||||||
|
|
||||||
it('pager works in headless mode with UI attached', function()
|
it('pager works in headless mode with UI attached', function()
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user