mirror of
https://github.com/vim/vim.git
synced 2025-09-23 03:43:49 -04:00
patch 8.2.4209: partial in 'opfunc' cannot use an imported function
Problem: partial in 'opfunc' cannot use an imported function. Solution: Also expand the function name in a partial. (closes #9614)
This commit is contained in:
@@ -4680,27 +4680,44 @@ copy_callback(callback_T *dest, callback_T *src)
|
|||||||
void
|
void
|
||||||
expand_autload_callback(callback_T *cb)
|
expand_autload_callback(callback_T *cb)
|
||||||
{
|
{
|
||||||
|
char_u *name;
|
||||||
char_u *p;
|
char_u *p;
|
||||||
imported_T *import;
|
imported_T *import;
|
||||||
|
|
||||||
if (!in_vim9script() || cb->cb_name == NULL || !cb->cb_free_name)
|
if (!in_vim9script() || cb->cb_name == NULL
|
||||||
|
|| (!cb->cb_free_name
|
||||||
|
&& (cb->cb_partial == NULL || cb->cb_partial->pt_name == NULL)))
|
||||||
return;
|
return;
|
||||||
p = vim_strchr(cb->cb_name, '.');
|
if (cb->cb_partial != NULL)
|
||||||
|
name = cb->cb_partial->pt_name;
|
||||||
|
else
|
||||||
|
name = cb->cb_name;
|
||||||
|
p = vim_strchr(name, '.');
|
||||||
if (p == NULL)
|
if (p == NULL)
|
||||||
return;
|
return;
|
||||||
import = find_imported(cb->cb_name, p - cb->cb_name, FALSE, NULL);
|
import = find_imported(name, p - name, FALSE, NULL);
|
||||||
if (import != NULL && SCRIPT_ID_VALID(import->imp_sid))
|
if (import != NULL && SCRIPT_ID_VALID(import->imp_sid))
|
||||||
{
|
{
|
||||||
scriptitem_T *si = SCRIPT_ITEM(import->imp_sid);
|
scriptitem_T *si = SCRIPT_ITEM(import->imp_sid);
|
||||||
|
|
||||||
if (si->sn_autoload_prefix != NULL)
|
if (si->sn_autoload_prefix != NULL)
|
||||||
{
|
{
|
||||||
char_u *name = concat_str(si->sn_autoload_prefix, p + 1);
|
char_u *newname = concat_str(si->sn_autoload_prefix, p + 1);
|
||||||
|
|
||||||
if (name != NULL)
|
if (newname != NULL)
|
||||||
{
|
{
|
||||||
vim_free(cb->cb_name);
|
if (cb->cb_partial != NULL)
|
||||||
cb->cb_name = name;
|
{
|
||||||
|
if (cb->cb_name == cb->cb_partial->pt_name)
|
||||||
|
cb->cb_name = newname;
|
||||||
|
vim_free(cb->cb_partial->pt_name);
|
||||||
|
cb->cb_partial->pt_name = newname;
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
vim_free(cb->cb_name);
|
||||||
|
cb->cb_name = newname;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@@ -673,6 +673,39 @@ def Test_use_autoload_import_in_insert_completion()
|
|||||||
&rtp = save_rtp
|
&rtp = save_rtp
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
|
def Test_use_autoload_import_partial_in_opfunc()
|
||||||
|
mkdir('Xdir/autoload', 'p')
|
||||||
|
var save_rtp = &rtp
|
||||||
|
exe 'set rtp^=' .. getcwd() .. '/Xdir'
|
||||||
|
|
||||||
|
var lines =<< trim END
|
||||||
|
vim9script
|
||||||
|
export def Opfunc(..._)
|
||||||
|
g:opfunc_called = 'yes'
|
||||||
|
enddef
|
||||||
|
END
|
||||||
|
writefile(lines, 'Xdir/autoload/opfunc.vim')
|
||||||
|
|
||||||
|
new
|
||||||
|
lines =<< trim END
|
||||||
|
vim9script
|
||||||
|
import autoload 'opfunc.vim'
|
||||||
|
nnoremap <expr> <F3> TheFunc()
|
||||||
|
def TheFunc(): string
|
||||||
|
&operatorfunc = function('opfunc.Opfunc', [0])
|
||||||
|
return 'g@'
|
||||||
|
enddef
|
||||||
|
feedkeys("\<F3>l", 'xt')
|
||||||
|
assert_equal('yes', g:opfunc_called)
|
||||||
|
END
|
||||||
|
CheckScriptSuccess(lines)
|
||||||
|
|
||||||
|
set opfunc=
|
||||||
|
bwipe!
|
||||||
|
delete('Xdir', 'rf')
|
||||||
|
&rtp = save_rtp
|
||||||
|
enddef
|
||||||
|
|
||||||
def Test_use_autoload_import_in_fold_expression()
|
def Test_use_autoload_import_in_fold_expression()
|
||||||
mkdir('Xdir/autoload', 'p')
|
mkdir('Xdir/autoload', 'p')
|
||||||
var save_rtp = &rtp
|
var save_rtp = &rtp
|
||||||
|
@@ -750,6 +750,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 */
|
||||||
|
/**/
|
||||||
|
4209,
|
||||||
/**/
|
/**/
|
||||||
4208,
|
4208,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user