patch 9.0.1607: screenpos() returns wrong row with diff filler lines
Problem: screenpos() returns wrong row with diff filler lines.
Solution: Only add filler lines when appropriate. Also don't add the
'smoothscroll' marker when w_skipcol is zero. (closes #12485,
closes #12484)
diff --git a/src/move.c b/src/move.c
index 54f5648..e021d71 100644
--- a/src/move.c
+++ b/src/move.c
@@ -344,7 +344,8 @@
check_topline = TRUE;
else if (check_top_offset())
check_topline = TRUE;
- else if (curwin->w_cursor.lnum == curwin->w_topline)
+ else if (curwin->w_skipcol > 0
+ && curwin->w_cursor.lnum == curwin->w_topline)
{
colnr_T vcol;
@@ -1459,7 +1460,8 @@
#ifdef FEAT_DIFF
// Add filler lines above this buffer line.
- row += diff_check_fill(wp, lnum);
+ row += lnum == wp->w_topline ? wp->w_topfill
+ : diff_check_fill(wp, lnum);
#endif
colnr_T off = win_col_off(wp);
@@ -1479,7 +1481,7 @@
col += off;
width = wp->w_width - off + win_col_off2(wp);
- if (pos->lnum == wp->w_topline)
+ if (lnum == wp->w_topline)
col -= wp->w_skipcol;
// long line wrapping, adjust row
@@ -1848,6 +1850,7 @@
}
}
+ // TODO: is comparing w_topline with prev_topline still needed?
if (curwin->w_topline == prev_topline
|| curwin->w_skipcol != prev_skipcol)
// need to redraw because wl_size of the topline may now be invalid