]> granicus.if.org Git - vim/commitdiff
patch 8.2.1271: Vim9: Error for Funcref function argument type v8.2.1271
authorBram Moolenaar <Bram@vim.org>
Wed, 22 Jul 2020 18:16:11 +0000 (20:16 +0200)
committerBram Moolenaar <Bram@vim.org>
Wed, 22 Jul 2020 18:16:11 +0000 (20:16 +0200)
Problem:    Vim9: Error for Funcref function argument type.
Solution:   Find the actual function type if possible. (issue #6507)

src/testdir/test_vim9_func.vim
src/version.c
src/vim9compile.c

index 3fa5d6abea4d30817eaadbbaa017e7011793cf14..ed5972185b81df9638c13cfb1a48797aebaf8562 100644 (file)
@@ -270,6 +270,19 @@ def Test_call_funcref()
     assert_equal(123, Funcref())
   END
   CheckScriptSuccess(lines)
+
+  lines =<< trim END
+    vim9script
+    def RetNumber(): number
+      return 123
+    enddef
+    def Bar(F: func: number): number
+      return F()
+    enddef
+    let Funcref = function('RetNumber')
+    assert_equal(123, Bar(Funcref))
+  END
+  CheckScriptSuccess(lines)
 enddef
 
 let SomeFunc = function('len')
index 9ac0340a300cdaa9fc2d335117b7fd5dabaec2a4..0bb6b0a75e313025976c6014ddadcd768cbf29b2 100644 (file)
@@ -754,6 +754,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1271,
 /**/
     1270,
 /**/
index 55f08a73ea83e47a8429512b5317fde49aaad418..999833b5221d530d8c4469a5d0731bbb0a581596 100644 (file)
@@ -599,6 +599,28 @@ check_argtype(type_T *expected, typval_T *actual_tv)
        member.tt_member = &t_any;
        actual.tt_member = &member;
     }
+    else if (actual_tv->v_type == VAR_FUNC || actual_tv->v_type == VAR_PARTIAL)
+    {
+       char_u  *name = NULL;
+       ufunc_T *ufunc = NULL;
+
+       if (actual_tv->v_type == VAR_PARTIAL)
+       {
+           if (actual_tv->vval.v_partial->pt_func != NULL)
+               ufunc = actual_tv->vval.v_partial->pt_func;
+           else
+               name = actual_tv->vval.v_partial->pt_name;
+       }
+       else
+           name = actual_tv->vval.v_string;
+       if (name != NULL)
+           // TODO: how about a builtin function?
+           ufunc = find_func(name, FALSE, NULL);
+       if (ufunc != NULL && ufunc->uf_func_type != NULL)
+           actual = *ufunc->uf_func_type;
+       else
+           actual.tt_member = &t_any;
+    }
     else
        actual.tt_member = &t_any;
     return check_type(expected, &actual, TRUE);