Jason Franklin wrote:
> I discovered that this patch introduces the following bug:
>
> 1. vim --clean
> 2. :set cul rnu
> 3. o1<CR>2<CR>3<CR><ESC>
> 4. y3k}
>
> Notice that the 'cursorline' highlighting was not updated properly.
>
> I was able to fix the problem with the following patch:
>
>
> diff --git a/src/move.c b/src/move.c
> index 404b2350f..b384bb703 100644
> --- a/src/move.c
> +++ b/src/move.c
> @@ -144,7 +144,7 @@ redraw_for_cursorline(win_T *wp)
> {
> if (wp->w_p_rnu)
> // win_line() will redraw the number column only.
> - redraw_win_later(wp, VALID);
> + redraw_win_later(wp, SOME_VALID);
> #ifdef FEAT_SYN_HL
> if (wp->w_p_cul)
> {
>
>
> The above patch reverts the behavior of redraw_for_cursorline() when
> 'relativenumber' is set. Previously, having 'rnu' set would result in
> the use of SOME_VALID in the redraw function call.
>
> Note that this fixes the problem for me, but I'm not sure if this takes
> away some of the performance improvements of this patch.
Thanks for looking into it. It's not the best solution. The problem is
that the w_last_cursorline is not updated when the window is redrawn.
Let me fix that.
--
-rwxr-xr-x 1 root 24 Oct 29 1929 /bin/ed
-rwxr-xr-t 4 root 131720 Jan 1 1970 /usr/ucb/vi
-rwxr-xr-x 1 root 5.89824e37 Oct 22 1990 /usr/bin/emacs
/// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
