Skip to content

Commit

Permalink
patch 8.2.3826: Vim9: using "g:Func" as funcref doesn't work in :def …
Browse files Browse the repository at this point in the history
…function

Problem:    Vim9: using "g:Func" as a funcref does not work in a :def
            function.
Solution:   Include "g:" in the function name. (closes #9336)
  • Loading branch information
brammool committed Dec 16, 2021
1 parent 52797ba commit b15cf44
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/testdir/test_vim9_disassemble.vim
Original file line number Diff line number Diff line change
Expand Up @@ -2413,7 +2413,7 @@ def Test_disassemble_dict_stack()
assert_match('<SNR>\d*_UseMember\_s*' ..
'var d = {func: Legacy}\_s*' ..
'\d PUSHS "func"\_s*' ..
'\d PUSHFUNC "Legacy"\_s*' ..
'\d PUSHFUNC "g:Legacy"\_s*' ..
'\d NEWDICT size 1\_s*' ..
'\d STORE $0\_s*' ..

Expand Down
12 changes: 12 additions & 0 deletions src/testdir/test_vim9_func.vim
Original file line number Diff line number Diff line change
Expand Up @@ -1232,11 +1232,23 @@ def Test_set_opfunc_to_global_function()
g:result = getreg('"')->count(' ')
return ''
enddef
# global function works at script level
&operatorfunc = g:CountSpaces
new
'a b c d e'->setline(1)
feedkeys("g@_", 'x')
assert_equal(4, g:result)

&operatorfunc = ''
g:result = 0
# global function works in :def function
def Func()
&operatorfunc = g:CountSpaces
enddef
Func()
feedkeys("g@_", 'x')
assert_equal(4, g:result)

bwipe!
END
CheckScriptSuccess(lines)
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Original file line number Diff line number Diff line change
Expand Up @@ -749,6 +749,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
3826,
/**/
3825,
/**/
Expand Down
16 changes: 15 additions & 1 deletion src/vim9compile.c
Original file line number Diff line number Diff line change
Expand Up @@ -1281,12 +1281,26 @@ generate_PUSHBLOB(cctx_T *cctx, blob_T *blob)
generate_PUSHFUNC(cctx_T *cctx, char_u *name, type_T *type)
{
isn_T *isn;
char_u *funcname;

RETURN_OK_IF_SKIP(cctx);
if ((isn = generate_instr_type(cctx, ISN_PUSHFUNC, type)) == NULL)
return FAIL;
isn->isn_arg.string = name == NULL ? NULL : vim_strsave(name);
if (name == NULL)
funcname = NULL;
else if (*name == K_SPECIAL) // script-local
funcname = vim_strsave(name);
else
{
funcname = alloc(STRLEN(name) + 3);
if (funcname != NULL)
{
STRCPY(funcname, "g:");
STRCPY(funcname + 2, name);
}
}

isn->isn_arg.string = funcname;
return OK;
}

Expand Down

0 comments on commit b15cf44

Please sign in to comment.