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