From a732c253b71f89702285d5ec6fd7803045f6add9 Mon Sep 17 00:00:00 2001 From: Dundar Goc Date: Sat, 7 May 2022 12:53:37 +0200 Subject: refactor: change type of linenr_T from long to int32_t The size of long varies depending on architecture, in contrast to the MAXLNUM constant which sets the maximum allowable number of lines to 2^32-1. This discrepancy may lead to hard to detect bugs, for example https://github.com/neovim/neovim/issues/18454. Setting linenr_T to a fix maximum size of 2^32-1 will prevent this type of errors in the future. Also change the variables `amount` and `amount_after` to be linenr_T since they're referring to "the line number difference" between two texts. --- src/nvim/terminal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/nvim/terminal.c') diff --git a/src/nvim/terminal.c b/src/nvim/terminal.c index c5b3bbcb01..a0adf482e4 100644 --- a/src/nvim/terminal.c +++ b/src/nvim/terminal.c @@ -1511,7 +1511,7 @@ static void adjust_scrollback(Terminal *term, buf_T *buf) term->sb_current--; xfree(term->sb_buffer[term->sb_current]); } - deleted_lines(1, (long)diff); + deleted_lines(1, (linenr_T)diff); } // Resize the scrollback storage. -- cgit