vim-patch:7.4.856

Problem:    "zt" still doesn't work well with filler lines. (Gary Johnson)
Solution:   Check for filler lines above the cursor. (Christian Brabandt)

https://github.com/vim/vim/commit/a09a2c5857ab854f0870573b5160da1964c905a2

2 files changed, 5 insertions, 8 deletions

diff --git a/src/nvim/move.c b/src/nvim/move.c
index 64f7b27cf2..ba79c0411a 100644
--- a/src/nvim/move.c
+++ b/src/nvim/move.c
@@ -1302,13 +1302,10 @@ void scroll_cursor_top(int min_scroll, int always)
   }
   new_topline = top + 1;
 
-  // used already contains the number of filler lines above, don't add it
+  // "used" already contains the number of filler lines above, don't add it
   // again.
-  // TODO: if filler lines above new top are to be considered as context for
-  // the current window, leave next statement commented, else hide filler
-  // lines above cursor line, by adding them to extra
-  // int extra += diff_check_fill(curwin, curwin->w_cursor.lnum);
-  int extra = 0;
+  // Hide filler lines above cursor line by adding them to "extra".
+  int extra = diff_check_fill(curwin, curwin->w_cursor.lnum);
 
   /*
    * Check if the lines from "top" to "bot" fit in the window.  If they do,
@@ -1317,7 +1314,7 @@ void scroll_cursor_top(int min_scroll, int always)
   while (top > 0) {
     int i = hasFolding(top, &top, NULL)
             ? 1  // count one logical line for a sequence of folded lines
-            : plines(top);
+            : plines_nofill(top);
     used += i;
     if (extra + i <= off && bot < curbuf->b_ml.ml_line_count) {
       if (hasFolding(bot, NULL, &bot))
diff --git a/src/nvim/version.c b/src/nvim/version.c
index 74b45cea53..f9aa99eb5c 100644
--- a/src/nvim/version.c
+++ b/src/nvim/version.c
@@ -432,7 +432,7 @@ static int included_patches[] = {
   // 859,
   858,
   // 857,
-  // 856,
+  856,
   // 855 NA
   // 854 NA
   853,