patch 9.0.1571: RedrawingDisabled not used consistently
Problem: RedrawingDisabled not used consistently.
Solution: Avoid RedrawingDisabled going negative. Set RedrawingDisabled in
win_split_ins(). (closes #11961)
diff --git a/src/ex_cmds.c b/src/ex_cmds.c
index 0fd6d10..20d4d9a 100644
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3219,7 +3219,8 @@
(void)keymap_init();
#endif
- --RedrawingDisabled;
+ if (RedrawingDisabled > 0)
+ --RedrawingDisabled;
did_inc_redrawing_disabled = FALSE;
if (!skip_redraw)
{
@@ -3263,7 +3264,7 @@
#endif
theend:
- if (did_inc_redrawing_disabled)
+ if (did_inc_redrawing_disabled && RedrawingDisabled > 0)
--RedrawingDisabled;
#if defined(FEAT_EVAL)
if (did_set_swapcommand)
@@ -3735,7 +3736,6 @@
int sublen;
int got_quit = FALSE;
int got_match = FALSE;
- int temp;
int which_pat;
char_u *cmd;
int save_State;
@@ -4316,7 +4316,7 @@
#endif
// Invert the matched string.
// Remove the inversion afterwards.
- temp = RedrawingDisabled;
+ int save_RedrawingDisabled = RedrawingDisabled;
RedrawingDisabled = 0;
// avoid calling update_screen() in vgetorpeek()
@@ -4386,7 +4386,7 @@
msg_scroll = i;
showruler(TRUE);
windgoto(msg_row, msg_col);
- RedrawingDisabled = temp;
+ RedrawingDisabled = save_RedrawingDisabled;
#ifdef USE_ON_FLY_SCROLL
dont_scroll = FALSE; // allow scrolling here