patch 9.0.0745: wrong cursor position when using "gj" and "gk" in a long line
Problem: Wrong cursor position when using "gj" and "gk" in a long line.
Solution: Adjust computations for the cursor position and skipcol. Re-enable
tests that pass now, disable failing breakindent test.
diff --git a/src/move.c b/src/move.c
index 343c6c8..995301e 100644
--- a/src/move.c
+++ b/src/move.c
@@ -1044,6 +1044,7 @@
colnr_T prev_skipcol;
long so = get_scrolloff_value();
long siso = get_sidescrolloff_value();
+ int did_sub_skipcol = FALSE;
/*
* First make sure that w_topline is valid (after moving the cursor).
@@ -1104,15 +1105,14 @@
// skip columns that are not visible
if (curwin->w_cursor.lnum == curwin->w_topline
&& curwin->w_wcol >= curwin->w_skipcol)
+ {
curwin->w_wcol -= curwin->w_skipcol;
+ did_sub_skipcol = TRUE;
+ }
// long line wrapping, adjust curwin->w_wrow
if (curwin->w_wcol >= curwin->w_width)
{
-#ifdef FEAT_LINEBREAK
- char_u *sbr;
-#endif
-
// this same formula is used in validate_cursor_col()
n = (curwin->w_wcol - curwin->w_width) / width + 1;
curwin->w_wcol -= n * width;
@@ -1122,7 +1122,7 @@
// When cursor wraps to first char of next line in Insert
// mode, the 'showbreak' string isn't shown, backup to first
// column
- sbr = get_showbreak_value(curwin);
+ char_u *sbr = get_showbreak_value(curwin);
if (*sbr && *ml_get_cursor() == NUL
&& curwin->w_wcol == vim_strsize(sbr))
curwin->w_wcol = 0;
@@ -1247,7 +1247,7 @@
if ((colnr_T)n >= curwin->w_height + curwin->w_skipcol / width - so)
extra += 2;
- if (extra == 3 || p_lines <= so * 2)
+ if (extra == 3 || curwin->w_height <= so * 2)
{
// not enough room for 'scrolloff', put cursor in the middle
n = curwin->w_virtcol / width;
@@ -1279,12 +1279,15 @@
while (endcol > curwin->w_virtcol)
endcol -= width;
if (endcol > curwin->w_skipcol)
- {
- curwin->w_wrow -= (endcol - curwin->w_skipcol) / width;
curwin->w_skipcol = endcol;
- }
}
+ // adjust w_wrow for the changed w_skipcol
+ if (did_sub_skipcol)
+ curwin->w_wrow -= (curwin->w_skipcol - prev_skipcol) / width;
+ else
+ curwin->w_wrow -= curwin->w_skipcol / width;
+
if (curwin->w_wrow >= curwin->w_height)
{
// small window, make sure cursor is in it