commit 346823d3e5668b99d2c2fd920e7f215e21ad3ea7
author Luuk van Baal <luukvbaal@gmail.com> Wed Oct 05 18:26:42 2022 +0100
committer Bram Moolenaar <Bram@vim.org> Wed Oct 05 18:26:42 2022 +0100
tree ab5d401f50309df98b3975d1d71365e838a2208a
parent 7beaf6a720ddc7e2989c8831872bfb98ec78a65d

patch 9.0.0667: ml_get error when 'splitkeep' is "screen"

Problem:    ml_get error when 'splitkeep' is "screen". (Marius Gedminas)
Solution:   Check the botline is not too large. (Luuk van Baal,
            closes #11293, closes #11292)

src/testdir/test_window_cmd.vim

diff --git a/src/testdir/test_window_cmd.vim b/src/testdir/test_window_cmd.vim
index 7718757..61daa8d 100644
--- a/src/testdir/test_window_cmd.vim
+++ b/src/testdir/test_window_cmd.vim
@@ -1807,6 +1807,13 @@
   let top = line('w0')
   help | quit
   call assert_equal(top, line('w0'))
+  " No error when resizing window in autocmd and buffer length changed
+  autocmd FileType qf exe "resize" line('$')
+  cexpr getline(1, '$')
+  copen
+  wincmd p
+  norm dd
+  cexpr getline(1, '$')
 
   %bwipeout!
   set splitbelow&

src/version.c

diff --git a/src/version.c b/src/version.c
index 925cb22..0d3e911 100644
--- a/src/version.c
+++ b/src/version.c
@@ -700,6 +700,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    667,
+/**/
     666,
 /**/
     665,

src/window.c

diff --git a/src/window.c b/src/window.c
index 79ff5a4..1f3f9ad 100644
--- a/src/window.c
+++ b/src/window.c
@@ -6356,7 +6356,8 @@
 	if (wp->w_height != wp->w_prev_height)
 	{
 	    // If window has moved update botline to keep the same screenlines.
-	    if (*p_spk == 's' && wp->w_winrow != wp->w_prev_winrow)
+	    if (*p_spk == 's' && wp->w_winrow != wp->w_prev_winrow
+		      && wp->w_botline - 1 <= wp->w_buffer->b_ml.ml_line_count)
 	    {
 		lnum = wp->w_cursor.lnum;
 		diff = (wp->w_winrow - wp->w_prev_winrow)