patch 9.1.0804: tests: no error check when setting global 'cc'
Problem: tests: no error check when setting global 'cc'
Solution: also parse and check global 'cc' value (Milly)
closes: #15914
Signed-off-by: Milly <milly.ca@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
diff --git a/src/buffer.c b/src/buffer.c
index 90be301..5359527 100644
--- a/src/buffer.c
+++ b/src/buffer.c
@@ -214,7 +214,7 @@
enter_buffer(curbuf);
#ifdef FEAT_SYN_HL
if (old_tw != curbuf->b_p_tw)
- check_colorcolumn(curwin);
+ check_colorcolumn(NULL, curwin);
#endif
return FAIL;
}
@@ -1213,7 +1213,7 @@
#ifdef FEAT_SYN_HL
if (old_tw != curbuf->b_p_tw)
- check_colorcolumn(curwin);
+ check_colorcolumn(NULL, curwin);
#endif
}
// If "old_curbuf" is NULL we are in big trouble here...
@@ -1911,7 +1911,7 @@
enter_buffer(buf);
#ifdef FEAT_SYN_HL
if (old_tw != curbuf->b_p_tw)
- check_colorcolumn(curwin);
+ check_colorcolumn(NULL, curwin);
#endif
}
}
diff --git a/src/option.c b/src/option.c
index a108c52..a55288a 100644
--- a/src/option.c
+++ b/src/option.c
@@ -4328,7 +4328,7 @@
tabpage_T *tp;
FOR_ALL_TAB_WINDOWS(tp, wp)
- check_colorcolumn(wp);
+ check_colorcolumn(NULL, wp);
}
#endif
@@ -6755,7 +6755,7 @@
#endif
#ifdef FEAT_SYN_HL
fill_culopt_flags(NULL, wp);
- check_colorcolumn(wp);
+ check_colorcolumn(NULL, wp);
#endif
set_listchars_option(wp, wp->w_p_lcs, TRUE, NULL, 0);
set_fillchars_option(wp, wp->w_p_fcs, TRUE, NULL, 0);
diff --git a/src/optionstr.c b/src/optionstr.c
index 2049633..1794746 100644
--- a/src/optionstr.c
+++ b/src/optionstr.c
@@ -1478,9 +1478,11 @@
* The 'colorcolumn' option is changed.
*/
char *
-did_set_colorcolumn(optset_T *args UNUSED)
+did_set_colorcolumn(optset_T *args)
{
- return check_colorcolumn(curwin);
+ char_u **varp = (char_u **)args->os_varp;
+
+ return check_colorcolumn(*varp, varp == &curwin->w_p_cc ? curwin : NULL);
}
#endif
diff --git a/src/proto/window.pro b/src/proto/window.pro
index 441070e..4ab7103 100644
--- a/src/proto/window.pro
+++ b/src/proto/window.pro
@@ -98,7 +98,7 @@
int win_hasvertsplit(void);
int get_win_number(win_T *wp, win_T *first_win);
int get_tab_number(tabpage_T *tp);
-char *check_colorcolumn(win_T *wp);
+char *check_colorcolumn(char_u *cc, win_T *wp);
int get_last_winid(void);
int win_locked(win_T *wp);
/* vim: set ft=c : */
diff --git a/src/testdir/gen_opt_test.vim b/src/testdir/gen_opt_test.vim
index 120ea9b..7e89e73 100644
--- a/src/testdir/gen_opt_test.vim
+++ b/src/testdir/gen_opt_test.vim
@@ -46,7 +46,6 @@
\ iminsert: 'The global value is always overwritten by the local value',
\ imsearch: 'The global value is always overwritten by the local value',
\ breakindentopt: 'TODO: fix missing error handling for setglobal',
- \ colorcolumn: 'TODO: fix missing error handling for setglobal',
\ conceallevel: 'TODO: fix missing error handling for setglobal',
\ foldcolumn: 'TODO: fix missing error handling for setglobal',
\ numberwidth: 'TODO: fix missing error handling for setglobal',
diff --git a/src/version.c b/src/version.c
index b26e848..59260d1 100644
--- a/src/version.c
+++ b/src/version.c
@@ -705,6 +705,8 @@
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 804,
+/**/
803,
/**/
802,
diff --git a/src/window.c b/src/window.c
index db907f8..5a8dc2e 100644
--- a/src/window.c
+++ b/src/window.c
@@ -7925,23 +7925,38 @@
}
/*
- * Handle setting 'colorcolumn' or 'textwidth' in window "wp".
+ * Check "cc" as 'colorcolumn' and update the members of "wp".
+ * This is called when 'colorcolumn' or 'textwidth' is changed.
* Returns error message, NULL if it's OK.
*/
char *
-check_colorcolumn(win_T *wp)
+check_colorcolumn(
+ char_u *cc, // when NULL: use "wp->w_p_cc"
+ win_T *wp) // when NULL: only parse "cc"
{
char_u *s;
+ int tw;
int col;
int count = 0;
int color_cols[256];
int i;
int j = 0;
- if (wp->w_buffer == NULL)
+ if (wp != NULL && wp->w_buffer == NULL)
return NULL; // buffer was closed
- for (s = wp->w_p_cc; *s != NUL && count < 255;)
+ if (cc != NULL)
+ s = cc;
+ else
+ s = wp->w_p_cc;
+
+ if (wp != NULL)
+ tw = wp->w_buffer->b_p_tw;
+ else
+ // buffer-local value not set, assume zero
+ tw = 0;
+
+ while (*s != NUL && count < 255)
{
if (*s == '-' || *s == '+')
{
@@ -7951,9 +7966,9 @@
if (!VIM_ISDIGIT(*s))
return e_invalid_argument;
col = col * getdigits(&s);
- if (wp->w_buffer->b_p_tw == 0)
+ if (tw == 0)
goto skip; // 'textwidth' not set, skip this item
- col += wp->w_buffer->b_p_tw;
+ col += tw;
if (col < 0)
goto skip;
}
@@ -7971,6 +7986,9 @@
return e_invalid_argument; // illegal trailing comma as in "set cc=80,"
}
+ if (wp == NULL)
+ return NULL; // only parse "cc"
+
vim_free(wp->w_p_cc_cols);
if (count == 0)
wp->w_p_cc_cols = NULL;