Patch 9.0.1328
Problem: Error when using "none" for GUI color is confusing.
Solution: Mention that the name should perhaps be "NONE". (closes #1400)
Files: src/errors.h, src/gui.c, src/testdir/test_highlight.vim
*** ../vim-9.0.1327/src/errors.h 2023-02-17 21:08:46.433731097 +0000
--- src/errors.h 2023-02-19 20:37:56.532793255 +0000
***************
*** 3447,3449 ****
--- 3447,3451 ----
EXTERN char e_using_null_object[]
INIT(= N_("E1360: Using a null object"));
#endif
+ EXTERN char e_cannot_use_color_none_did_you_mean_none[]
+ INIT(= N_("E1361: Cannot use color \"none\", did you mean \"NONE\"?"));
*** ../vim-9.0.1327/src/gui.c 2023-01-08 13:44:21.065352366 +0000
--- src/gui.c 2023-02-19 20:46:27.579963938 +0000
***************
*** 4634,4645 ****
return INVALCOLOR;
t = gui_mch_get_color(name);
if (t == INVALCOLOR
#if defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK)
! && gui.in_use
#endif
)
! semsg(_(e_cannot_allocate_color_str), name);
return t;
}
--- 4634,4651 ----
return INVALCOLOR;
t = gui_mch_get_color(name);
+ int is_none = STRCMP(name, "none") == 0;
if (t == INVALCOLOR
#if defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK)
! && (gui.in_use || is_none)
#endif
)
! {
! if (is_none)
! emsg(_(e_cannot_use_color_none_did_you_mean_none));
! else
! semsg(_(e_cannot_allocate_color_str), name);
! }
return t;
}
*** ../vim-9.0.1327/src/testdir/test_highlight.vim 2022-09-28
21:06:30.634345977 +0100
--- src/testdir/test_highlight.vim 2023-02-19 20:48:02.903872527 +0000
***************
*** 46,51 ****
--- 46,55 ----
call assert_equal("Group3 xxx cleared",
\ split(execute("hi Group3"), "\n")[0])
call assert_fails("hi Crash term='asdf", "E475:")
+
+ if has('gui_running')
+ call assert_fails('hi NotUsed guibg=none', 'E1361:')
+ endif
endfunc
func HighlightArgs(name)
*** ../vim-9.0.1327/src/version.c 2023-02-19 18:36:24.727281727 +0000
--- src/version.c 2023-02-19 20:41:54.620317587 +0000
***************
*** 697,698 ****
--- 697,700 ----
{ /* Add new patch number below this line */
+ /**/
+ 1328,
/**/
--
hundred-and-one symptoms of being an internet addict:
152. You find yourself falling for someone you've never seen or hardly
know, but, boy can he/she TYPE!!!!!!
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
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].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/20230219205036.B164B1C07A0%40moolenaar.net.