patch 9.0.0697: cursor in wrong position with Visual substitute
Problem: Cursor in wrong position with Visual substitute.
Solution: When restoring 'linebreak' mark the virtual column as invalid.
(closes #11309, closes #11311)
diff --git a/src/ops.c b/src/ops.c
index 99a5d5c..3e641f4 100644
--- a/src/ops.c
+++ b/src/ops.c
@@ -2208,6 +2208,35 @@
}
/*
+ * Reset 'linebreak' and take care of side effects.
+ * Returns the previous value, to be passed to restore_lbr().
+ */
+ static int
+reset_lbr(void)
+{
+ if (!curwin->w_p_lbr)
+ return FALSE;
+ // changing 'linebreak' may require w_virtcol to be updated
+ curwin->w_p_lbr = FALSE;
+ curwin->w_valid &= ~(VALID_WROW|VALID_WCOL|VALID_VIRTCOL);
+ return TRUE;
+}
+
+/*
+ * Restore 'linebreak' and take care of side effects.
+ */
+ static void
+restore_lbr(int lbr_saved)
+{
+ if (!curwin->w_p_lbr && lbr_saved)
+ {
+ // changing 'linebreak' may require w_virtcol to be updated
+ curwin->w_p_lbr = TRUE;
+ curwin->w_valid &= ~(VALID_WROW|VALID_WCOL|VALID_VIRTCOL);
+ }
+}
+
+/*
* prepare a few things for block mode yank/delete/tilde
*
* for delete:
@@ -2235,11 +2264,10 @@
char_u *prev_pend;
chartabsize_T cts;
#ifdef FEAT_LINEBREAK
- int lbr_saved = curwin->w_p_lbr;
-
- // Avoid a problem with unwanted linebreaks in block mode.
- curwin->w_p_lbr = FALSE;
+ // Avoid a problem with unwanted linebreaks in block mode.
+ int lbr_saved = reset_lbr();
#endif
+
bdp->startspaces = 0;
bdp->endspaces = 0;
bdp->textlen = 0;
@@ -2369,7 +2397,7 @@
bdp->textcol = (colnr_T) (pstart - line);
bdp->textstart = pstart;
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
}
@@ -3570,9 +3598,7 @@
#ifdef FEAT_LINEBREAK
// Avoid a problem with unwanted linebreaks in block mode.
- if (curwin->w_p_lbr)
- curwin->w_valid &= ~VALID_VIRTCOL;
- curwin->w_p_lbr = FALSE;
+ (void)reset_lbr();
#endif
oap->is_VIsual = VIsual_active;
if (oap->motion_force == 'V')
@@ -3908,7 +3934,7 @@
{
#ifdef FEAT_LINEBREAK
// make sure redrawing is correct
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
redraw_curbuf_later(UPD_INVERTED);
}
@@ -3948,7 +3974,7 @@
))
{
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
redraw_curbuf_later(UPD_INVERTED);
}
@@ -4036,7 +4062,7 @@
else
{
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
oap->excl_tr_ws = cap->cmdchar == 'z';
(void)op_yank(oap, FALSE, !gui_yank);
@@ -4065,7 +4091,7 @@
#ifdef FEAT_LINEBREAK
// Restore linebreak, so that when the user edits it looks as
// before.
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
// Reset finish_op now, don't want it set inside edit().
finish_op = FALSE;
@@ -4143,7 +4169,7 @@
#ifdef FEAT_LINEBREAK
// Restore linebreak, so that when the user edits it looks as
// before.
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
// call 'operatorfunc'
op_function(oap);
@@ -4172,12 +4198,12 @@
#ifdef FEAT_LINEBREAK
// Restore linebreak, so that when the user edits it looks as
// before.
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
op_insert(oap, cap->count1);
#ifdef FEAT_LINEBREAK
// Reset linebreak, so that formatting works correctly.
- curwin->w_p_lbr = FALSE;
+ (void)reset_lbr();
#endif
// TODO: when inserting in several lines, should format all
@@ -4203,7 +4229,7 @@
#ifdef FEAT_LINEBREAK
// Restore linebreak, so that when the user edits it looks as
// before.
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
op_replace(oap, cap->nchar);
}
@@ -4246,7 +4272,7 @@
{
VIsual_active = TRUE;
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
op_addsub(oap, cap->count1, redo_VIsual.rv_arg);
VIsual_active = FALSE;
@@ -4265,7 +4291,7 @@
|| oap->op_type == OP_DELETE))
{
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = FALSE;
+ (void)reset_lbr();
#endif
coladvance(curwin->w_curswant = old_col);
}
@@ -4279,6 +4305,6 @@
motion_force = NUL;
}
#ifdef FEAT_LINEBREAK
- curwin->w_p_lbr = lbr_saved;
+ restore_lbr(lbr_saved);
#endif
}