Dominique Pellé wrote:

> Please try the attached patch.
> It fixes the pum bug described in this thread.
>
> Bug is in function 'validate_cursor()' which means that
> features other than the pum may be affected by the fix.
> So patch needs to be tested and reviewed.
>
> My previous email described how to reproduce the bug.

I tested further and still found an issue which is
addressed in this new attached patch.

Regards
-- Dominique
-- 
You received this message from the "vim_use" maillist.
For more information, visit http://www.vim.org/maillist.php
diff -r fb1222c880fc src/move.c
--- a/src/move.c	Wed Jan 20 21:41:47 2010 +0100
+++ b/src/move.c	Wed Jan 20 23:47:01 2010 +0100
@@ -901,10 +901,8 @@
 	if (curwin->w_p_wrap
 		&& col >= (colnr_T)W_WIDTH(curwin)
 		&& W_WIDTH(curwin) - off + curwin_col_off2() > 0)
-	{
-	    col -= W_WIDTH(curwin);
-	    col = col % (W_WIDTH(curwin) - off + curwin_col_off2());
-	}
+	    col = (col - off)
+	       % (W_WIDTH(curwin) - off + curwin_col_off2()) + off;
 	if (col > (int)curwin->w_leftcol)
 	    col -= curwin->w_leftcol;
 	else

Reply via email to