forked from aniani/vim
patch 8.2.5167: get(Fn, 'name') on funcref returns special byte code
Problem: get(Fn, 'name') on funcref returns special byte code. Solution: Use the printable name.
This commit is contained in:
@@ -4719,19 +4719,23 @@ f_get(typval_T *argvars, typval_T *rettv)
|
|||||||
if (pt != NULL)
|
if (pt != NULL)
|
||||||
{
|
{
|
||||||
char_u *what = tv_get_string(&argvars[1]);
|
char_u *what = tv_get_string(&argvars[1]);
|
||||||
char_u *n;
|
|
||||||
|
|
||||||
if (STRCMP(what, "func") == 0 || STRCMP(what, "name") == 0)
|
if (STRCMP(what, "func") == 0 || STRCMP(what, "name") == 0)
|
||||||
{
|
{
|
||||||
|
char_u *name = partial_name(pt);
|
||||||
|
|
||||||
rettv->v_type = (*what == 'f' ? VAR_FUNC : VAR_STRING);
|
rettv->v_type = (*what == 'f' ? VAR_FUNC : VAR_STRING);
|
||||||
n = partial_name(pt);
|
if (name == NULL)
|
||||||
if (n == NULL)
|
|
||||||
rettv->vval.v_string = NULL;
|
rettv->vval.v_string = NULL;
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
rettv->vval.v_string = vim_strsave(n);
|
|
||||||
if (rettv->v_type == VAR_FUNC)
|
if (rettv->v_type == VAR_FUNC)
|
||||||
func_ref(rettv->vval.v_string);
|
func_ref(name);
|
||||||
|
if (*what == 'n' && pt->pt_name == NULL
|
||||||
|
&& pt->pt_func != NULL)
|
||||||
|
// use <SNR> instead of the byte code
|
||||||
|
name = printable_func_name(pt->pt_func);
|
||||||
|
rettv->vval.v_string = vim_strsave(name);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
else if (STRCMP(what, "dict") == 0)
|
else if (STRCMP(what, "dict") == 0)
|
||||||
|
@@ -134,11 +134,20 @@ func Test_get_lambda()
|
|||||||
call assert_equal([], get(l:L, 'args'))
|
call assert_equal([], get(l:L, 'args'))
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
|
func s:FooBar()
|
||||||
|
endfunc
|
||||||
|
|
||||||
" get({func}, {what} [, {default}])
|
" get({func}, {what} [, {default}])
|
||||||
func Test_get_func()
|
func Test_get_func()
|
||||||
let l:F = function('tr')
|
let l:F = function('tr')
|
||||||
call assert_equal('tr', get(l:F, 'name'))
|
call assert_equal('tr', get(l:F, 'name'))
|
||||||
call assert_equal(l:F, get(l:F, 'func'))
|
call assert_equal(l:F, get(l:F, 'func'))
|
||||||
|
|
||||||
|
let Fb_func = function('s:FooBar')
|
||||||
|
call assert_match('<SNR>\d\+_FooBar', get(Fb_func, 'name'))
|
||||||
|
let Fb_ref = funcref('s:FooBar')
|
||||||
|
call assert_match('<SNR>\d\+_FooBar', get(Fb_ref, 'name'))
|
||||||
|
|
||||||
call assert_equal({'func has': 'no dict'}, get(l:F, 'dict', {'func has': 'no dict'}))
|
call assert_equal({'func has': 'no dict'}, get(l:F, 'dict', {'func has': 'no dict'}))
|
||||||
call assert_equal(0, get(l:F, 'dict'))
|
call assert_equal(0, get(l:F, 'dict'))
|
||||||
call assert_equal([], get(l:F, 'args'))
|
call assert_equal([], get(l:F, 'args'))
|
||||||
|
@@ -735,6 +735,8 @@ static char *(features[]) =
|
|||||||
|
|
||||||
static int included_patches[] =
|
static int included_patches[] =
|
||||||
{ /* Add new patch number below this line */
|
{ /* Add new patch number below this line */
|
||||||
|
/**/
|
||||||
|
5167,
|
||||||
/**/
|
/**/
|
||||||
5166,
|
5166,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user