aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-07-26 11:04:33 -04:00
Code

patch 8.0.1072: :highlight command causes a redraw even when nothing changed

Browse Source 
Problem:    The :highlight command causes a redraw even when nothing changed.
Solution:   Only set "need_highlight_changed" when an attribute changed.
This commit is contained in:
Bram Moolenaar 2017-09-08 12:37:47 +02:00
parent c6da01a5b8
commit 99433291b1
2 changed files with 109 additions and 86 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

193
src/syntax.c
View File

@ -7364,6 +7364,8 @@ do_highlight(
int attr; int attr;
int id; int id;
int idx; int idx;
struct hl_group *item;
struct hl_group item_before;
int dodefault = FALSE; int dodefault = FALSE;
int doclear = FALSE; int doclear = FALSE;
int dolink = FALSE; int dolink = FALSE;
@ -7459,12 +7461,13 @@ do_highlight(
} }
from_id = syn_check_group(from_start, (int)(from_end - from_start)); from_id = syn_check_group(from_start, (int)(from_end - from_start));
item = &HL_TABLE()[from_id - 1];
if (STRNCMP(to_start, "NONE", 4) == 0) if (STRNCMP(to_start, "NONE", 4) == 0)
to_id = 0; to_id = 0;
else else
to_id = syn_check_group(to_start, (int)(to_end - to_start)); to_id = syn_check_group(to_start, (int)(to_end - to_start));
if (from_id > 0 && (!init || HL_TABLE()[from_id - 1].sg_set == 0)) if (from_id > 0 && (!init || item->sg_set == 0))
{ {
/* /*
* Don't allow a link when there already is some highlighting * Don't allow a link when there already is some highlighting
@ -7476,22 +7479,26 @@ do_highlight(
if (sourcing_name == NULL && !dodefault) if (sourcing_name == NULL && !dodefault)
EMSG(_("E414: group has settings, highlight link ignored")); EMSG(_("E414: group has settings, highlight link ignored"));
} }
else else if (item->sg_link != to_id
#ifdef FEAT_EVAL
|| item->sg_scriptID != current_SID
#endif
|| item->sg_cleared)
{ {
if (!init) if (!init)
HL_TABLE()[from_id - 1].sg_set |= SG_LINK; item->sg_set |= SG_LINK;
HL_TABLE()[from_id - 1].sg_link = to_id; item->sg_link = to_id;
#ifdef FEAT_EVAL #ifdef FEAT_EVAL
HL_TABLE()[from_id - 1].sg_scriptID = current_SID; item->sg_scriptID = current_SID;
#endif #endif
HL_TABLE()[from_id - 1].sg_cleared = FALSE; item->sg_cleared = FALSE;
redraw_all_later(SOME_VALID); redraw_all_later(SOME_VALID);
/* Only call highlight_changed() once after multiple changes. */
need_highlight_changed = TRUE;
} }
} }
/* Only call highlight_changed() once, after sourcing a syntax file */
need_highlight_changed = TRUE;
return; return;
} }
@ -7577,19 +7584,23 @@ do_highlight(
if (id == 0) /* failed (out of memory) */ if (id == 0) /* failed (out of memory) */
return; return;
idx = id - 1; /* index is ID minus one */ idx = id - 1; /* index is ID minus one */
item = &HL_TABLE()[idx];
/* Return if "default" was used and the group already has settings. */ /* Return if "default" was used and the group already has settings. */
if (dodefault && hl_has_settings(idx, TRUE)) if (dodefault && hl_has_settings(idx, TRUE))
return; return;
if (STRCMP(HL_TABLE()[idx].sg_name_u, "NORMAL") == 0) /* Make a copy so we can check if any attribute actually changed. */
item_before = *item;
if (STRCMP(item->sg_name_u, "NORMAL") == 0)
is_normal_group = TRUE; is_normal_group = TRUE;
#ifdef FEAT_GUI_X11 #ifdef FEAT_GUI_X11
else if (STRCMP(HL_TABLE()[idx].sg_name_u, "MENU") == 0) else if (STRCMP(item->sg_name_u, "MENU") == 0)
is_menu_group = TRUE; is_menu_group = TRUE;
else if (STRCMP(HL_TABLE()[idx].sg_name_u, "SCROLLBAR") == 0) else if (STRCMP(item->sg_name_u, "SCROLLBAR") == 0)
is_scrollbar_group = TRUE; is_scrollbar_group = TRUE;
else if (STRCMP(HL_TABLE()[idx].sg_name_u, "TOOLTIP") == 0) else if (STRCMP(item->sg_name_u, "TOOLTIP") == 0)
is_tooltip_group = TRUE; is_tooltip_group = TRUE;
#endif #endif
@ -7598,7 +7609,7 @@ do_highlight(
{ {
highlight_clear(idx); highlight_clear(idx);
if (!doclear) if (!doclear)
HL_TABLE()[idx].sg_set = 0; item->sg_set = 0;
} }
if (!doclear) if (!doclear)
@ -7629,10 +7640,10 @@ do_highlight(
if (STRCMP(key, "NONE") == 0) if (STRCMP(key, "NONE") == 0)
{ {
if (!init || HL_TABLE()[idx].sg_set == 0) if (!init || item->sg_set == 0)
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_TERM+SG_CTERM+SG_GUI; item->sg_set |= SG_TERM+SG_CTERM+SG_GUI;
highlight_clear(idx); highlight_clear(idx);
} }
continue; continue;
@ -7719,31 +7730,31 @@ do_highlight(
break; break;
if (*key == 'T') if (*key == 'T')
{ {
if (!init || !(HL_TABLE()[idx].sg_set & SG_TERM)) if (!init || !(item->sg_set & SG_TERM))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_TERM; item->sg_set |= SG_TERM;
HL_TABLE()[idx].sg_term = attr; item->sg_term = attr;
} }
} }
else if (*key == 'C') else if (*key == 'C')
{ {
if (!init || !(HL_TABLE()[idx].sg_set & SG_CTERM)) if (!init || !(item->sg_set & SG_CTERM))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_CTERM; item->sg_set |= SG_CTERM;
HL_TABLE()[idx].sg_cterm = attr; item->sg_cterm = attr;
HL_TABLE()[idx].sg_cterm_bold = FALSE; item->sg_cterm_bold = FALSE;
} }
} }
#if defined(FEAT_GUI) || defined(FEAT_EVAL) #if defined(FEAT_GUI) || defined(FEAT_EVAL)
else else
{ {
if (!init || !(HL_TABLE()[idx].sg_set & SG_GUI)) if (!init || !(item->sg_set & SG_GUI))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_GUI; item->sg_set |= SG_GUI;
HL_TABLE()[idx].sg_gui = attr; item->sg_gui = attr;
} }
} }
#endif #endif
@ -7752,69 +7763,74 @@ do_highlight(
{ {
/* in non-GUI fonts are simply ignored */ /* in non-GUI fonts are simply ignored */
#ifdef FEAT_GUI #ifdef FEAT_GUI
if (!gui.shell_created) if (item->sg_font_name != NULL
&& STRCMP(item->sg_font_name, arg) == 0)
{
/* Font name didn't change, ignore. */
}
else if (!gui.shell_created)
{ {
/* GUI not started yet, always accept the name. */ /* GUI not started yet, always accept the name. */
vim_free(HL_TABLE()[idx].sg_font_name); vim_free(item->sg_font_name);
HL_TABLE()[idx].sg_font_name = vim_strsave(arg); item->sg_font_name = vim_strsave(arg);
} }
else else
{ {
GuiFont temp_sg_font = HL_TABLE()[idx].sg_font; GuiFont temp_sg_font = item->sg_font;
# ifdef FEAT_XFONTSET # ifdef FEAT_XFONTSET
GuiFontset temp_sg_fontset = HL_TABLE()[idx].sg_fontset; GuiFontset temp_sg_fontset = item->sg_fontset;
# endif # endif
/* First, save the current font/fontset. /* First, save the current font/fontset.
* Then try to allocate the font/fontset. * Then try to allocate the font/fontset.
* If the allocation fails, HL_TABLE()[idx].sg_font OR * If the allocation fails, item->sg_font OR
* sg_fontset will be set to NOFONT or NOFONTSET respectively. * sg_fontset will be set to NOFONT or NOFONTSET respectively.
*/ */
HL_TABLE()[idx].sg_font = NOFONT; item->sg_font = NOFONT;
# ifdef FEAT_XFONTSET # ifdef FEAT_XFONTSET
HL_TABLE()[idx].sg_fontset = NOFONTSET; item->sg_fontset = NOFONTSET;
# endif # endif
hl_do_font(idx, arg, is_normal_group, is_menu_group, hl_do_font(idx, arg, is_normal_group, is_menu_group,
is_tooltip_group, FALSE); is_tooltip_group, FALSE);
# ifdef FEAT_XFONTSET # ifdef FEAT_XFONTSET
if (HL_TABLE()[idx].sg_fontset != NOFONTSET) if (item->sg_fontset != NOFONTSET)
{ {
/* New fontset was accepted. Free the old one, if there /* New fontset was accepted. Free the old one, if there
* was one. */ * was one. */
gui_mch_free_fontset(temp_sg_fontset); gui_mch_free_fontset(temp_sg_fontset);
vim_free(HL_TABLE()[idx].sg_font_name); vim_free(item->sg_font_name);
HL_TABLE()[idx].sg_font_name = vim_strsave(arg); item->sg_font_name = vim_strsave(arg);
} }
else else
HL_TABLE()[idx].sg_fontset = temp_sg_fontset; item->sg_fontset = temp_sg_fontset;
# endif # endif
if (HL_TABLE()[idx].sg_font != NOFONT) if (item->sg_font != NOFONT)
{ {
/* New font was accepted. Free the old one, if there was /* New font was accepted. Free the old one, if there was
* one. */ * one. */
gui_mch_free_font(temp_sg_font); gui_mch_free_font(temp_sg_font);
vim_free(HL_TABLE()[idx].sg_font_name); vim_free(item->sg_font_name);
HL_TABLE()[idx].sg_font_name = vim_strsave(arg); item->sg_font_name = vim_strsave(arg);
} }
else else
HL_TABLE()[idx].sg_font = temp_sg_font; item->sg_font = temp_sg_font;
} }
#endif #endif
} }
else if (STRCMP(key, "CTERMFG") == 0 || STRCMP(key, "CTERMBG") == 0) else if (STRCMP(key, "CTERMFG") == 0 || STRCMP(key, "CTERMBG") == 0)
{ {
if (!init || !(HL_TABLE()[idx].sg_set & SG_CTERM)) if (!init || !(item->sg_set & SG_CTERM))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_CTERM; item->sg_set |= SG_CTERM;
/* When setting the foreground color, and previously the "bold" /* When setting the foreground color, and previously the "bold"
* flag was set for a light color, reset it now */ * flag was set for a light color, reset it now */
if (key[5] == 'F' && HL_TABLE()[idx].sg_cterm_bold) if (key[5] == 'F' && item->sg_cterm_bold)
{ {
HL_TABLE()[idx].sg_cterm &= ~HL_BOLD; item->sg_cterm &= ~HL_BOLD;
HL_TABLE()[idx].sg_cterm_bold = FALSE; item->sg_cterm_bold = FALSE;
} }
if (VIM_ISDIGIT(*arg)) if (VIM_ISDIGIT(*arg))
@ -7871,22 +7887,22 @@ do_highlight(
* colors (on some terminals, e.g. "linux") */ * colors (on some terminals, e.g. "linux") */
if (bold == TRUE) if (bold == TRUE)
{ {
HL_TABLE()[idx].sg_cterm |= HL_BOLD; item->sg_cterm |= HL_BOLD;
HL_TABLE()[idx].sg_cterm_bold = TRUE; item->sg_cterm_bold = TRUE;
} }
else if (bold == FALSE) else if (bold == FALSE)
HL_TABLE()[idx].sg_cterm &= ~HL_BOLD; item->sg_cterm &= ~HL_BOLD;
} }
/* Add one to the argument, to avoid zero. Zero is used for /* Add one to the argument, to avoid zero. Zero is used for
* "NONE", then "color" is -1. */ * "NONE", then "color" is -1. */
if (key[5] == 'F') if (key[5] == 'F')
{ {
HL_TABLE()[idx].sg_cterm_fg = color + 1; item->sg_cterm_fg = color + 1;
if (is_normal_group) if (is_normal_group)
{ {
cterm_normal_fg_color = color + 1; cterm_normal_fg_color = color + 1;
cterm_normal_fg_bold = (HL_TABLE()[idx].sg_cterm & HL_BOLD); cterm_normal_fg_bold = (item->sg_cterm & HL_BOLD);
#ifdef FEAT_GUI #ifdef FEAT_GUI
/* Don't do this if the GUI is used. */ /* Don't do this if the GUI is used. */
if (!gui.in_use && !gui.starting) if (!gui.in_use && !gui.starting)
@ -7900,7 +7916,7 @@ do_highlight(
} }
else else
{ {
HL_TABLE()[idx].sg_cterm_bg = color + 1; item->sg_cterm_bg = color + 1;
if (is_normal_group) if (is_normal_group)
{ {
cterm_normal_bg_color = color + 1; cterm_normal_bg_color = color + 1;
@ -7940,23 +7956,23 @@ do_highlight(
else if (STRCMP(key, "GUIFG") == 0) else if (STRCMP(key, "GUIFG") == 0)
{ {
#if defined(FEAT_GUI) || defined(FEAT_EVAL) #if defined(FEAT_GUI) || defined(FEAT_EVAL)
if (!init || !(HL_TABLE()[idx].sg_set & SG_GUI)) if (!init || !(item->sg_set & SG_GUI))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_GUI; item->sg_set |= SG_GUI;
# if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS) # if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS)
/* In GUI guifg colors are only used when recognized */ /* In GUI guifg colors are only used when recognized */
i = color_name2handle(arg); i = color_name2handle(arg);
if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !USE_24BIT) if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !USE_24BIT)
{ {
HL_TABLE()[idx].sg_gui_fg = i; item->sg_gui_fg = i;
# endif # endif
vim_free(HL_TABLE()[idx].sg_gui_fg_name); vim_free(item->sg_gui_fg_name);
if (STRCMP(arg, "NONE") != 0) if (STRCMP(arg, "NONE") != 0)
HL_TABLE()[idx].sg_gui_fg_name = vim_strsave(arg); item->sg_gui_fg_name = vim_strsave(arg);
else else
HL_TABLE()[idx].sg_gui_fg_name = NULL; item->sg_gui_fg_name = NULL;
# if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS) # if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS)
# ifdef FEAT_GUI_X11 # ifdef FEAT_GUI_X11
if (is_menu_group) if (is_menu_group)
@ -7977,23 +7993,23 @@ do_highlight(
else if (STRCMP(key, "GUIBG") == 0) else if (STRCMP(key, "GUIBG") == 0)
{ {
#if defined(FEAT_GUI) || defined(FEAT_EVAL) #if defined(FEAT_GUI) || defined(FEAT_EVAL)
if (!init || !(HL_TABLE()[idx].sg_set & SG_GUI)) if (!init || !(item->sg_set & SG_GUI))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_GUI; item->sg_set |= SG_GUI;
# if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS) # if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS)
/* In GUI guifg colors are only used when recognized */ /* In GUI guifg colors are only used when recognized */
i = color_name2handle(arg); i = color_name2handle(arg);
if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !USE_24BIT) if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !USE_24BIT)
{ {
HL_TABLE()[idx].sg_gui_bg = i; item->sg_gui_bg = i;
# endif # endif
vim_free(HL_TABLE()[idx].sg_gui_bg_name); vim_free(item->sg_gui_bg_name);
if (STRCMP(arg, "NONE") != 0) if (STRCMP(arg, "NONE") != 0)
HL_TABLE()[idx].sg_gui_bg_name = vim_strsave(arg); item->sg_gui_bg_name = vim_strsave(arg);
else else
HL_TABLE()[idx].sg_gui_bg_name = NULL; item->sg_gui_bg_name = NULL;
# if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS) # if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS)
# ifdef FEAT_GUI_X11 # ifdef FEAT_GUI_X11
if (is_menu_group) if (is_menu_group)
@ -8014,22 +8030,22 @@ do_highlight(
else if (STRCMP(key, "GUISP") == 0) else if (STRCMP(key, "GUISP") == 0)
{ {
#if defined(FEAT_GUI) || defined(FEAT_EVAL) #if defined(FEAT_GUI) || defined(FEAT_EVAL)
if (!init || !(HL_TABLE()[idx].sg_set & SG_GUI)) if (!init || !(item->sg_set & SG_GUI))
{ {
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_GUI; item->sg_set |= SG_GUI;
# ifdef FEAT_GUI # ifdef FEAT_GUI
i = color_name2handle(arg); i = color_name2handle(arg);
if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !gui.in_use) if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !gui.in_use)
{ {
HL_TABLE()[idx].sg_gui_sp = i; item->sg_gui_sp = i;
# endif # endif
vim_free(HL_TABLE()[idx].sg_gui_sp_name); vim_free(item->sg_gui_sp_name);
if (STRCMP(arg, "NONE") != 0) if (STRCMP(arg, "NONE") != 0)
HL_TABLE()[idx].sg_gui_sp_name = vim_strsave(arg); item->sg_gui_sp_name = vim_strsave(arg);
else else
HL_TABLE()[idx].sg_gui_sp_name = NULL; item->sg_gui_sp_name = NULL;
# ifdef FEAT_GUI # ifdef FEAT_GUI
} }
# endif # endif
@ -8042,7 +8058,7 @@ do_highlight(
char_u *tname; char_u *tname;
if (!init) if (!init)
HL_TABLE()[idx].sg_set |= SG_TERM; item->sg_set |= SG_TERM;
/* /*
* The "start" and "stop" arguments can be a literal escape * The "start" and "stop" arguments can be a literal escape
@ -8109,13 +8125,13 @@ do_highlight(
p = vim_strsave(buf); p = vim_strsave(buf);
if (key[2] == 'A') if (key[2] == 'A')
{ {
vim_free(HL_TABLE()[idx].sg_start); vim_free(item->sg_start);
HL_TABLE()[idx].sg_start = p; item->sg_start = p;
} }
else else
{ {
vim_free(HL_TABLE()[idx].sg_stop); vim_free(item->sg_stop);
HL_TABLE()[idx].sg_stop = p; item->sg_stop = p;
} }
} }
else else
@ -8124,13 +8140,13 @@ do_highlight(
error = TRUE; error = TRUE;
break; break;
} }
HL_TABLE()[idx].sg_cleared = FALSE; item->sg_cleared = FALSE;
/* /*
* When highlighting has been given for a group, don't link it. * When highlighting has been given for a group, don't link it.
*/ */
if (!init || !(HL_TABLE()[idx].sg_set & SG_LINK)) if (!init || !(item->sg_set & SG_LINK))
HL_TABLE()[idx].sg_link = 0; item->sg_link = 0;
/* /*
* Continue with next argument. * Continue with next argument.
@ -8147,10 +8163,10 @@ do_highlight(
{ {
if (is_normal_group) if (is_normal_group)
{ {
HL_TABLE()[idx].sg_term_attr = 0; item->sg_term_attr = 0;
HL_TABLE()[idx].sg_cterm_attr = 0; item->sg_cterm_attr = 0;
#ifdef FEAT_GUI #ifdef FEAT_GUI
HL_TABLE()[idx].sg_gui_attr = 0; item->sg_gui_attr = 0;
/* /*
* Need to update all groups, because they might be using "bg" * Need to update all groups, because they might be using "bg"
* and/or "fg", which have been changed now. * and/or "fg", which have been changed now.
@ -8185,15 +8201,20 @@ do_highlight(
else else
set_hl_attr(idx); set_hl_attr(idx);
#ifdef FEAT_EVAL #ifdef FEAT_EVAL
HL_TABLE()[idx].sg_scriptID = current_SID; item->sg_scriptID = current_SID;
#endif #endif
redraw_all_later(NOT_VALID);
} }
vim_free(key); vim_free(key);
vim_free(arg); vim_free(arg);
/* Only call highlight_changed() once, after sourcing a syntax file */ /* Only call highlight_changed() once, after a sequence of highlight
need_highlight_changed = TRUE; * commands, and only if an attribute actually changed. */
if (memcmp(item, &item_before, sizeof(item_before)) != 0)
{
redraw_all_later(NOT_VALID);
need_highlight_changed = TRUE;
}
} }
#if defined(EXITFREE) || defined(PROTO) #if defined(EXITFREE) || defined(PROTO)

2
src/version.c
View File

@ -769,6 +769,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 */
/**/
1072,
/**/ /**/
1071, 1071,
/**/ /**/