patch 8.1.0233: "safe" argument of call_vim_function() is always FALSE

Problem:    "safe" argument of call_vim_function() is always FALSE.
Solution:   Remove the argument.

diff --git a/src/edit.c b/src/edit.c
index b0b44e606cd5710dc91c14c574266e772df552de..daadc7b7b078945bbfc81745a2e873cbe2b9e088 100644 (file)
--- a/src/edit.c
+++ b/src/edit.c
@@ -4239,7 +4239,7 @@ expand_by_function(
     curbuf_save = curbuf;
 
     /* Call a function, which returns a list or dict. */
-    if (call_vim_function(funcname, 2, args, &rettv, FALSE) == OK)
+    if (call_vim_function(funcname, 2, args, &rettv) == OK)
     {
        switch (rettv.v_type)
        {
@@ -5569,7 +5569,7 @@ ins_complete(int c, int enable_pum)
            pos = curwin->w_cursor;
            curwin_save = curwin;
            curbuf_save = curbuf;
-           col = call_func_retnr(funcname, 2, args, FALSE);
+           col = call_func_retnr(funcname, 2, args);
            if (curwin_save != curwin || curbuf_save != curbuf)
            {
                EMSG(_(e_complwin));

diff --git a/src/eval.c b/src/eval.c
index 6034cd24f74d8b609d4b33ba18b3f4184d43797f..3a275213cbf8653677be5061f5c03308e764c04d 100644 (file)
--- a/src/eval.c
+++ b/src/eval.c
@@ -1021,29 +1021,15 @@ call_vim_function(
     char_u      *func,
     int                argc,
     typval_T   *argv,
-    typval_T   *rettv,
-    int                safe)           /* use the sandbox */
+    typval_T   *rettv)
 {
     int                doesrange;
-    void       *save_funccalp = NULL;
     int                ret;
 
-    if (safe)
-    {
-       save_funccalp = save_funccal();
-       ++sandbox;
-    }
-
     rettv->v_type = VAR_UNKNOWN;               /* clear_tv() uses this */
     ret = call_func(func, (int)STRLEN(func), rettv, argc, argv, NULL,
                    curwin->w_cursor.lnum, curwin->w_cursor.lnum,
                    &doesrange, TRUE, NULL, NULL);
-    if (safe)
-    {
-       --sandbox;
-       restore_funccal(save_funccalp);
-    }
-
     if (ret == FAIL)
        clear_tv(rettv);
 
@@ -1060,13 +1046,12 @@ call_vim_function(
 call_func_retnr(
     char_u      *func,
     int                argc,
-    typval_T   *argv,
-    int                safe)           /* use the sandbox */
+    typval_T   *argv)
 {
     typval_T   rettv;
     varnumber_T        retval;
 
-    if (call_vim_function(func, argc, argv, &rettv, safe) == FAIL)
+    if (call_vim_function(func, argc, argv, &rettv) == FAIL)
        return -1;
 
     retval = get_tv_number_chk(&rettv, NULL);
@@ -1088,13 +1073,12 @@ call_func_retnr(
 call_func_retstr(
     char_u      *func,
     int                argc,
-    typval_T   *argv,
-    int                safe)           /* use the sandbox */
+    typval_T   *argv)
 {
     typval_T   rettv;
     char_u     *retval;
 
-    if (call_vim_function(func, argc, argv, &rettv, safe) == FAIL)
+    if (call_vim_function(func, argc, argv, &rettv) == FAIL)
        return NULL;
 
     retval = vim_strsave(get_tv_string(&rettv));
@@ -1113,12 +1097,11 @@ call_func_retstr(
 call_func_retlist(
     char_u      *func,
     int                argc,
-    typval_T   *argv,
-    int                safe)           /* use the sandbox */
+    typval_T   *argv)
 {
     typval_T   rettv;
 
-    if (call_vim_function(func, argc, argv, &rettv, safe) == FAIL)
+    if (call_vim_function(func, argc, argv, &rettv) == FAIL)
        return NULL;
 
     if (rettv.v_type != VAR_LIST)

diff --git a/src/ex_getln.c b/src/ex_getln.c
index e659c1a8c2e505c167dec6dd65a7d63f186d3542..54b37961f5157644ec59e44d4114a76c5827c57d 100644 (file)
--- a/src/ex_getln.c
+++ b/src/ex_getln.c
@@ -5279,7 +5279,7 @@ expand_shellcmd(
  */
     static void *
 call_user_expand_func(
-    void       *(*user_expand_func)(char_u *, int, typval_T *, int),
+    void       *(*user_expand_func)(char_u *, int, typval_T *),
     expand_T   *xp,
     int                *num_file,
     char_u     ***file)
@@ -5318,7 +5318,7 @@ call_user_expand_func(
     ccline.cmdprompt = NULL;
     current_SID = xp->xp_scriptID;
 
-    ret = user_expand_func(xp->xp_arg, 3, args, FALSE);
+    ret = user_expand_func(xp->xp_arg, 3, args);
 
     ccline = save_ccline;
     current_SID = save_current_SID;

diff --git a/src/mbyte.c b/src/mbyte.c
index 1f85246f0a4a62ca0eb34b7053eabea1c6dee958..96b39b4114c983cfd63a6f6a63d1e4225c183440 100644 (file)
--- a/src/mbyte.c
+++ b/src/mbyte.c
@@ -4825,7 +4825,7 @@ call_imactivatefunc(int active)
     argv[0].v_type = VAR_NUMBER;
     argv[0].vval.v_number = active ? 1 : 0;
     argv[1].v_type = VAR_UNKNOWN;
-    (void)call_func_retnr(p_imaf, 1, argv, FALSE);
+    (void)call_func_retnr(p_imaf, 1, argv);
 }
 
     static int
@@ -4839,7 +4839,7 @@ call_imstatusfunc(void)
     /* FIXME: :py print 'xxx' is shown duplicate result.
      * Use silent to avoid it. */
     ++msg_silent;
-    is_active = call_func_retnr(p_imsf, 0, NULL, FALSE);
+    is_active = call_func_retnr(p_imsf, 0, NULL);
     --msg_silent;
     return (is_active > 0);
 }

diff --git a/src/normal.c b/src/normal.c
index f3cd664a663876a11804a1f7889b943bded1ffb8..41c76233226a5ee4b9ac8d12fca42361624b7183 100644 (file)
--- a/src/normal.c
+++ b/src/normal.c
@@ -2248,7 +2248,7 @@ op_function(oparg_T *oap UNUSED)
        virtual_op = MAYBE;
 # endif
 
-       (void)call_func_retnr(p_opfunc, 1, argv, FALSE);
+       (void)call_func_retnr(p_opfunc, 1, argv);
 
 # ifdef FEAT_VIRTUALEDIT
        virtual_op = save_virtual_op;

diff --git a/src/proto/eval.pro b/src/proto/eval.pro
index 98f6656717a9ccfc9740c0e4014122b57183a88a..537b6490ad31d2ab2331ec58259d2dfcad592b4b 100644 (file)
--- a/src/proto/eval.pro
+++ b/src/proto/eval.pro
@@ -19,10 +19,10 @@ varnumber_T eval_to_number(char_u *expr);
 list_T *eval_spell_expr(char_u *badword, char_u *expr);
 int get_spellword(list_T *list, char_u **pp);
 typval_T *eval_expr(char_u *arg, char_u **nextcmd);
-int call_vim_function(char_u *func, int argc, typval_T *argv, typval_T *rettv, int safe);
-varnumber_T call_func_retnr(char_u *func, int argc, typval_T *argv, int safe);
-void *call_func_retstr(char_u *func, int argc, typval_T *argv, int safe);
-void *call_func_retlist(char_u *func, int argc, typval_T *argv, int safe);
+int call_vim_function(char_u *func, int argc, typval_T *argv, typval_T *rettv);
+varnumber_T call_func_retnr(char_u *func, int argc, typval_T *argv);
+void *call_func_retstr(char_u *func, int argc, typval_T *argv);
+void *call_func_retlist(char_u *func, int argc, typval_T *argv);
 int eval_foldexpr(char_u *arg, int *cp);
 void ex_let(exarg_T *eap);
 void list_hashtable_vars(hashtab_T *ht, char_u *prefix, int empty, int *first);

diff --git a/src/version.c b/src/version.c
index 64aa76abb1f698920c8a6e29621a786d7ddfa780..6e6c3cdc76effb3c2ff31664a9c4b0a5074acaaf 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -794,6 +794,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    233,
 /**/
     232,
 /**/