Hi,

2016/4/30 Sat 5:59:42 UTC+9 Bram Moolenaar wrote:
> Patch 7.4.1799
> Problem:    'guicolors' is a confusing option name.
> Solution:   Use 'termguicolors' instead. (Hirohito Higashi)
> Files:      runtime/doc/options.txt, runtime/doc/term.txt,
>             runtime/doc/various.txt, runtime/syntax/dircolors.vim, src/eval.c,
>             src/feature.h, src/globals.h, src/hardcopy.c, src/option.c,
>             src/option.h, src/proto/term.pro, src/screen.c, src/structs.h,
>             src/syntax.c, src/term.c, src/version.c, src/vim.h

>               /* In GUI guifg colors are only used when recognized */
>               i = color_name2handle(arg);
>               if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0
>   #  ifdef FEAT_GUI
>                       || !(USE_24BIT)
>   #  else
> !                     || !p_tgc
>   #  endif
>                  )
>               {

I think this ifdef is redundant.
Attached patch fixes this.

Regards,
Ken Takata

-- 
-- 
You received this message from the "vim_dev" 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_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.
# HG changeset patch
# Parent  27f652f3521a4d318acbc55c879ffb0a7d3d5c40

diff --git a/src/syntax.c b/src/syntax.c
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -7795,13 +7795,7 @@ do_highlight(
 # if defined(FEAT_GUI) || defined(FEAT_TERMGUICOLORS)
 		/* In GUI guifg colors are only used when recognized */
 		i = color_name2handle(arg);
-		if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0
-#  ifdef FEAT_GUI
-			|| !(USE_24BIT)
-#  else
-			|| !p_tgc
-#  endif
-		   )
+		if (i != INVALCOLOR || STRCMP(arg, "NONE") == 0 || !USE_24BIT)
 		{
 		    HL_TABLE()[idx].sg_gui_fg = i;
 # endif

Raspunde prin e-mail lui