On Mo, 09 Jun 2014, Bram Moolenaar wrote:

> Yes, it appears the "if settings already exist" only checks for term
> settings, not gui settings.  This is inconsistant.  I suppose we can
> change that without causing trouble for existing setups.

Here is a patch.

Best,
Christian
-- 
Wahrhaftigkeit ist die größte List.
                -- Georg Christoph Lichtenberg

-- 
-- 
You received this message from the "vim_use" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_use" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_use+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
diff --git a/src/syntax.c b/src/syntax.c
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -8045,6 +8045,10 @@ hl_has_settings(idx, check_link)
 	    || HL_TABLE()[idx].sg_cterm_attr != 0
 #ifdef FEAT_GUI
 	    || HL_TABLE()[idx].sg_gui_attr != 0
+	    || HL_TABLE()[idx].sg_gui_fg_name != NULL
+	    || HL_TABLE()[idx].sg_gui_bg_name != NULL
+	    || HL_TABLE()[idx].sg_gui_sp_name != NULL
+	    || HL_TABLE()[idx].sg_font_name != NUL
 #endif
 	    || (check_link && (HL_TABLE()[idx].sg_set & SG_LINK)));
 }

Reply via email to