mirror of
https://github.com/vim/vim.git
synced 2025-09-28 04:24:06 -04:00
patch 8.1.0021: clang warns for undefined behavior
Problem: Clang warns for undefined behavior. Solution: Move #ifdef outside of sprintf() call.(suggestion by Michael Jarvis, closes #2956)
This commit is contained in:
11
src/term.c
11
src/term.c
@@ -2872,14 +2872,15 @@ term_color(char_u *s, int n)
|
|||||||
#else
|
#else
|
||||||
char *format = "%s%s%%dm";
|
char *format = "%s%s%%dm";
|
||||||
#endif
|
#endif
|
||||||
sprintf(buf, format,
|
char *lead = i == 2 ? (
|
||||||
i == 2 ?
|
|
||||||
#if defined(FEAT_VTP) && defined(FEAT_TERMGUICOLORS)
|
#if defined(FEAT_VTP) && defined(FEAT_TERMGUICOLORS)
|
||||||
s[1] == '|' ? IF_EB("\033|", ESC_STR "|") :
|
s[1] == '|' ? IF_EB("\033|", ESC_STR "|") :
|
||||||
#endif
|
#endif
|
||||||
IF_EB("\033[", ESC_STR "[") : "\233",
|
IF_EB("\033[", ESC_STR "[")) : "\233";
|
||||||
s[i] == '3' ? (n >= 16 ? "38;5;" : "9")
|
char *tail = s[i] == '3' ? (n >= 16 ? "38;5;" : "9")
|
||||||
: (n >= 16 ? "48;5;" : "10"));
|
: (n >= 16 ? "48;5;" : "10");
|
||||||
|
|
||||||
|
sprintf(buf, format, lead, tail);
|
||||||
OUT_STR(tgoto(buf, 0, n >= 16 ? n : n - 8));
|
OUT_STR(tgoto(buf, 0, n >= 16 ? n : n - 8));
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
|
@@ -761,6 +761,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 */
|
||||||
|
/**/
|
||||||
|
21,
|
||||||
/**/
|
/**/
|
||||||
20,
|
20,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user