patch 8.2.4614: redrawing too much when 'cursorline' is set
Problem: Redrawing too much when 'cursorline' is set and jumping around.
Solution: Rely on win_update() to redraw the current and previous cursor
line, do not mark lines as modified. (closes #9996)
diff --git a/src/move.c b/src/move.c
index 1c88e6e..571ec8c 100644
--- a/src/move.c
+++ b/src/move.c
@@ -115,14 +115,6 @@
set_empty_rows(wp, done);
}
-#ifdef FEAT_SYN_HL
- void
-reset_cursorline(void)
-{
- curwin->w_last_cursorline = 0;
-}
-#endif
-
/*
* Redraw when w_cline_row changes and 'relativenumber' or 'cursorline' is
* set.
@@ -138,24 +130,8 @@
&& (wp->w_valid & VALID_CROW) == 0
&& !pum_visible())
{
- if (wp->w_p_rnu)
- // win_line() will redraw the number column only.
- redraw_win_later(wp, VALID);
-#ifdef FEAT_SYN_HL
- if (wp->w_p_cul)
- {
- if (wp->w_redr_type <= VALID && wp->w_last_cursorline != 0)
- {
- // "w_last_cursorline" may be outdated, worst case we redraw
- // too much. This is optimized for moving the cursor around in
- // the current window.
- redrawWinline(wp, wp->w_last_cursorline);
- redrawWinline(wp, wp->w_cursor.lnum);
- }
- else
- redraw_win_later(wp, SOME_VALID);
- }
-#endif
+ // win_line() will redraw the number column and cursorline only.
+ redraw_win_later(wp, VALID);
}
}