patch 9.0.1722: wrong error messages when passing wrong types to count()
Commit:
https://github.com/vim/vim/commit/4f389e7c0fe7dfeccfa512a72fa36f9028d57159
Author: zeertzjq <[email protected]>
Date: Thu Aug 17 22:10:40 2023 +0200
patch 9.0.1722: wrong error messages when passing wrong types to count()
Problem: wrong error messages when passing wrong types to count()
Solution: fix it
This fixes two problems:
1. When passing wrong type to {ic} argument of count(), two error
messages are given, the second of which is misleading.
2. When passing wrong type to {comp} argument of count(), the error
message doesn't mention that {comp} may be a String.
closes: #12825
Signed-off-by: Christian Brabandt <[email protected]>
Co-authored-by: zeertzjq <[email protected]>
diff --git a/src/errors.h b/src/errors.h
index 6ef47000a..1c1b146c9 100644
--- a/src/errors.h
+++ b/src/errors.h
@@ -1802,7 +1802,8 @@ EXTERN char
e_funcref_variable_name_must_start_with_capital_str[]
INIT(= N_("E704: Funcref variable name must start with a capital: %s"));
EXTERN char e_variable_name_conflicts_with_existing_function_str[]
INIT(= N_("E705: Variable name conflicts with existing function: %s"));
-// E706 unused
+EXTERN char e_argument_of_str_must_be_list_string_or_dictionary[]
+ INIT(= N_("E706: Argument of %s must be a List, String or Dictionary"));
EXTERN char e_function_name_conflicts_with_variable_str[]
INIT(= N_("E707: Function name conflicts with variable: %s"));
EXTERN char e_slice_must_come_last[]
diff --git a/src/list.c b/src/list.c
index d52c23dea..933480f55 100644
--- a/src/list.c
+++ b/src/list.c
@@ -2735,8 +2735,9 @@ f_count(typval_T *argvars, typval_T *rettv)
else
n = dict_count(argvars[0].vval.v_dict, &argvars[1], ic);
}
- else
- semsg(_(e_argument_of_str_must_be_list_or_dictionary), "count()");
+ else if (!error)
+ semsg(_(e_argument_of_str_must_be_list_string_or_dictionary),
+ "count()");
rettv->vval.v_number = n;
}
diff --git a/src/testdir/test_functions.vim b/src/testdir/test_functions.vim
index 750c4a2bf..e5d56ac18 100644
--- a/src/testdir/test_functions.vim
+++ b/src/testdir/test_functions.vim
@@ -1749,7 +1749,8 @@ func Test_count()
call assert_equal(2, count("fooooo", "oo"))
call assert_equal(0, count("foo", ""))
- call assert_fails('call count(0, 0)', 'E712:')
+ call assert_fails('call count(0, 0)', 'E706:')
+ call assert_fails('call count("", "", {})', ['E728:', 'E728:'])
endfunc
func Test_changenr()
diff --git a/src/version.c b/src/version.c
index c4098c2e7..1809d0d2b 100644
--- a/src/version.c
+++ b/src/version.c
@@ -695,6 +695,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
+/**/
+ 1722,
/**/
1721,
/**/
--
--
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/E1qWjOa-006Wca-Av%40256bit.org.
