From 1de62b9ea17b08db0fe37caf1f054b7b809120c3 Mon Sep 17 00:00:00 2001 From: zeertzjq Date: Sat, 13 Aug 2022 12:25:01 +0800 Subject: fix(charclass): make behavior with empty str match latest Vim (#19749) Later Vim patches changed to return 0 for empty string and null string. Also update setcellwidth() docs to match latest Vim. --- src/nvim/mbyte.c | 6 ++---- src/nvim/testdir/test_functions.vim | 2 ++ 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/nvim/mbyte.c b/src/nvim/mbyte.c index 378a08131d..53bbaab694 100644 --- a/src/nvim/mbyte.c +++ b/src/nvim/mbyte.c @@ -2861,10 +2861,8 @@ void f_setcellwidths(typval_T *argvars, typval_T *rettv, FunPtr fptr) void f_charclass(typval_T *argvars, typval_T *rettv, FunPtr fptr) { - if (argvars[0].v_type != VAR_STRING - || argvars[0].vval.v_string == NULL - || *argvars[0].vval.v_string == NUL) { - emsg(_(e_stringreq)); + if (tv_check_for_string(&argvars[0]) == FAIL + || argvars[0].vval.v_string == NULL) { return; } rettv->vval.v_number = mb_get_class((const char_u *)argvars[0].vval.v_string); diff --git a/src/nvim/testdir/test_functions.vim b/src/nvim/testdir/test_functions.vim index e0e0c1ca38..44b6f0373e 100644 --- a/src/nvim/testdir/test_functions.vim +++ b/src/nvim/testdir/test_functions.vim @@ -1774,6 +1774,8 @@ func Test_charclass() call assert_equal(1, charclass('.')) call assert_equal(2, charclass('x')) call assert_equal(3, charclass("\u203c")) + " this used to crash vim + call assert_equal(0, "xxx"[-1]->charclass()) endfunc func Test_eventhandler() -- cgit