thac0/vim
1
0
Fork 0
forked from aniani/vim
Code Activity

patch 8.2.3588: break statement is never reached

Browse Source 
Problem:    Break statement is never reached.
Solution:   Rely on return value of set_chars_option() not changing.
            (closes #9103)
This commit is contained in:
Bram Moolenaar
2021-11-12 19:52:47 +00:00
parent 0a15c7676b
commit 606efc7df4
2 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/optionstr.c
View File

@@ -1331,11 +1331,10 @@ ambw_end:
if (!(opt_flags & OPT_GLOBAL)) if (!(opt_flags & OPT_GLOBAL))
clear_string_option(&curwin->w_p_lcs); clear_string_option(&curwin->w_p_lcs);
FOR_ALL_TAB_WINDOWS(tp, wp) FOR_ALL_TAB_WINDOWS(tp, wp)
{ // If no error was returned above, we don't expect an error
errmsg = set_chars_option(wp, &wp->w_p_lcs); // here, so ignore the return value.
if (errmsg) (void)set_chars_option(wp, &wp->w_p_lcs);
break;
}
redraw_all_later(NOT_VALID); redraw_all_later(NOT_VALID);
} }
} }

2
src/version.c
View File

@@ -757,6 +757,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
3588,
/**/ /**/
3587, 3587,
/**/ /**/