To: vim_dev@googlegroups.com
Subject: Patch 9.0.1509
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
------------

Patch 9.0.1509
Problem:    Error message lacks mentioning the erroneous argument.
Solution:   Specify the argument that the error is for.
Files:      src/terminal.c, src/testdir/test_terminal.vim


*** ../vim-9.0.1508/src/terminal.c	2023-04-29 14:25:59.179203254 +0100
--- src/terminal.c	2023-05-05 22:04:46.783713632 +0100
***************
*** 6562,6568 ****
      if (argvars[1].vval.v_list->lv_first == &range_list_item
  	    || argvars[1].vval.v_list->lv_len != 16)
      {
! 	emsg(_(e_invalid_argument));
  	return;
      }
  
--- 6562,6568 ----
      if (argvars[1].vval.v_list->lv_first == &range_list_item
  	    || argvars[1].vval.v_list->lv_len != 16)
      {
! 	semsg(_(e_invalid_value_for_argument_str), "\"colors\"");
  	return;
      }
  
*** ../vim-9.0.1508/src/testdir/test_terminal.vim	2022-10-15 20:52:22.563752598 +0100
--- src/testdir/test_terminal.vim	2023-05-05 22:04:05.071647814 +0100
***************
*** 2188,2193 ****
--- 2188,2194 ----
    let colors[4] = 'Invalid'
    call assert_fails('call term_setansicolors(buf, colors)', 'E254:')
    call assert_fails('call term_setansicolors(buf, {})', 'E1211:')
+   call assert_fails('call term_setansicolors(buf, [])', 'E475: Invalid value for argument "colors"')
    set tgc&
  
    call StopShellInTerminal(buf)
*** ../vim-9.0.1508/src/version.c	2023-05-05 17:22:31.965659631 +0100
--- src/version.c	2023-05-05 22:02:53.943530202 +0100
***************
*** 697,698 ****
--- 697,700 ----
  {   /* Add new patch number below this line */
+ /**/
+     1509,
  /**/

-- 
It's totally unfair to suggest - as many have - that engineers are socially
inept.  Engineers simply have different objectives when it comes to social
interaction.
				(Scott Adams - The Dilbert principle)

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///                                                                      \\\
\\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///