GitHub/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim synced 2025-07-16 01:01:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(terminal): fix resize crash with pending scrollback (#14891)

Browse Source 
refresh_scrollback assumes pending scrollback rows exist only if the
terminal window height decreased (or the screen was full).

However, after accumulating scrollback, it's possible in some cases for
the terminal height to increase before refresh_scrollback is called via
invalidation (especially when the terminal buffer isn't initially
displayed in a window before nvim_open_term), which may crash.

As we'll have enough room for some scrollback rows, just append them to
the top of the buffer until it fills the window, then continue with the
previous logic for any remaining scrollback rows if necessary.
This commit is contained in:
Sean Dewar
2021-12-15 15:18:06 +00:00
committed by GitHub
parent cf65071e2e
commit 0a1391fdd7
2 changed files with 82 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

70
test/functional/terminal/scrollback_spec.lua
View File

@ -12,6 +12,8 @@ local curbufmeths = helpers.curbufmeths
local nvim = helpers.nvim
local feed_data = thelpers.feed_data
local pcall_err = helpers.pcall_err
local exec_lua = helpers.exec_lua
local assert_alive = helpers.assert_alive
describe(':terminal scrollback', function()
local screen
@ -527,3 +529,71 @@ describe("'scrollback' option", function()
end)
end)
describe("pending scrollback line handling", function()
local screen
before_each(function()
clear()
screen = Screen.new(30, 7)
screen:attach()
screen:set_default_attr_ids {
[1] = {foreground = Screen.colors.Brown},
[2] = {reverse = true},
[3] = {bold = true},
}
end)
it("does not crash after setting 'number' #14891", function()
exec_lua [[
local a = vim.api
local buf = a.nvim_create_buf(true, true)
local chan = a.nvim_open_term(buf, {})
a.nvim_win_set_option(0, "number", true)
a.nvim_chan_send(chan, ("a\n"):rep(11) .. "a")
a.nvim_win_set_buf(0, buf)
]]
screen:expect [[
{1: 1 }^a |
{1: 2 } a |
{1: 3 } a |
{1: 4 } a |
{1: 5 } a |
{1: 6 } a |
|
]]
feed('G')
screen:expect [[
{1: 7 } a |
{1: 8 } a |
{1: 9 } a |
{1: 10 } a |
{1: 11 } a |
{1: 12 } ^a |
|
]]
assert_alive()
end)
it("does not crash after nvim_buf_call #14891", function()
exec_lua [[
local a = vim.api
local bufnr = a.nvim_create_buf(false, true)
a.nvim_buf_call(bufnr, function()
vim.fn.termopen({"echo", ("hi\n"):rep(11)})
end)
a.nvim_win_set_buf(0, bufnr)
vim.cmd("startinsert")
]]
screen:expect [[
hi |
hi |
hi |
|
|
[Process exited 0]{2: } |
{3:-- TERMINAL --} |
]]
assert_alive()
end)
end)