]> granicus.if.org Git - vim/commitdiff
patch 9.0.0549: duplicated code in calling a :def function v9.0.0549
authorBram Moolenaar <Bram@vim.org>
Thu, 22 Sep 2022 16:34:01 +0000 (17:34 +0100)
committerBram Moolenaar <Bram@vim.org>
Thu, 22 Sep 2022 16:34:01 +0000 (17:34 +0100)
Problem:    Duplicated code in calling a :def function.
Solution:   Simplify the code.

src/version.c
src/vim9execute.c

index 2ad2658d321b2bb7bdedc4818746655766e69ded..e3a3a0e45a0d7c3ead7b799006fc1a51556c5493 100644 (file)
@@ -699,6 +699,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    549,
 /**/
     548,
 /**/
index 9416cf3a35ad13fd378c7e3096ddabc1af62b3ba..1699caf49640418fe2d9784b9585be578808343b 100644 (file)
@@ -380,7 +380,7 @@ get_pt_outer(partial_T *pt)
  * Call compiled function "cdf_idx" from compiled code.
  * This adds a stack frame and sets the instruction pointer to the start of the
  * called function.
- * If "pt" is not null use "pt->pt_outer" for ec_outer_ref->or_outer.
+ * If "pt_arg" is not NULL use "pt_arg->pt_outer" for ec_outer_ref->or_outer.
  *
  * Stack has:
  * - current arguments (already there)
@@ -394,7 +394,7 @@ get_pt_outer(partial_T *pt)
     static int
 call_dfunc(
        int             cdf_idx,
-       partial_T       *pt,
+       partial_T       *pt_arg,
        int             argcount_arg,
        ectx_T          *ectx)
 {
@@ -543,27 +543,21 @@ call_dfunc(
     STACK_TV_BOT(STACK_FRAME_IDX_OFF)->vval.v_number = ectx->ec_frame_idx;
     ectx->ec_frame_idx = ectx->ec_stack.ga_len;
 
-    // Initialize local variables
-    for (idx = 0; idx < dfunc->df_varcount; ++idx)
+    // Initialize all local variables to number zero.  Also initialize the
+    // variable that counts how many closures were created.  This is used in
+    // handle_closure_in_use().
+    int initcount = dfunc->df_varcount + (dfunc->df_has_closure ? 1 : 0);
+    for (idx = 0; idx < initcount; ++idx)
     {
        typval_T *tv = STACK_TV_BOT(STACK_FRAME_SIZE + idx);
 
        tv->v_type = VAR_NUMBER;
        tv->vval.v_number = 0;
     }
-    if (dfunc->df_has_closure)
-    {
-       typval_T *tv = STACK_TV_BOT(STACK_FRAME_SIZE + dfunc->df_varcount);
-
-       // Initialize the variable that counts how many closures were created.
-       // This is used in handle_closure_in_use().
-       tv->v_type = VAR_NUMBER;
-       tv->vval.v_number = 0;
-    }
     ectx->ec_stack.ga_len += STACK_FRAME_SIZE + varcount;
 
-    if (pt != NULL || ufunc->uf_partial != NULL
-                                            || (ufunc->uf_flags & FC_CLOSURE))
+    partial_T *pt = pt_arg != NULL ? pt_arg : ufunc->uf_partial;
+    if (pt != NULL || (ufunc->uf_flags & FC_CLOSURE))
     {
        outer_ref_T *ref = ALLOC_CLEAR_ONE(outer_ref_T);
 
@@ -575,12 +569,6 @@ call_dfunc(
            ++pt->pt_refcount;
            ref->or_partial = pt;
        }
-       else if (ufunc->uf_partial != NULL)
-       {
-           ref->or_outer = get_pt_outer(ufunc->uf_partial);
-           ++ufunc->uf_partial->pt_refcount;
-           ref->or_partial = ufunc->uf_partial;
-       }
        else
        {
            ref->or_outer = ALLOC_CLEAR_ONE(outer_T);
@@ -5832,7 +5820,9 @@ call_def_function(
     ectx.ec_where.wt_index = 0;
     ectx.ec_where.wt_variable = FALSE;
 
-    // Execute the instructions until done.
+    /*
+     * Execute the instructions until done.
+     */
     ret = exec_instructions(&ectx);
     if (ret == OK)
     {