# HG changeset patch # User Christian Brabandt # Date 1469559608 -7200 # Node ID f1920505bc16504f1302c2c66fdf328117c48437 # Parent 9335bbb35ee83c5421a3eb43bd7f403a854405f7 commit https://github.com/vim/vim/commit/97baee80f0906ee2f651ee1215ec033e84f866ad Author: Bram Moolenaar Date: Tue Jul 26 20:46:08 2016 +0200 patch 7.4.2104 Problem: Code duplication when unreferencing a function. Solution: De-duplicate. diff --git a/src/userfunc.c b/src/userfunc.c --- a/src/userfunc.c +++ b/src/userfunc.c @@ -2640,11 +2640,11 @@ ex_delfunction(exarg_T *eap) void func_unref(char_u *name) { - ufunc_T *fp; + ufunc_T *fp = NULL; if (name == NULL) return; - else if (isdigit(*name)) + if (isdigit(*name)) { fp = find_func(name); if (fp == NULL) @@ -2654,25 +2654,18 @@ func_unref(char_u *name) #endif EMSG2(_(e_intern2), "func_unref()"); } - else if (--fp->uf_refcount <= 0) - { - /* Only delete it when it's not being used. Otherwise it's done - * when "uf_calls" becomes zero. */ - if (fp->uf_calls == 0) - func_free(fp); - } } else if (STRNCMP(name, "", 8) == 0) { /* fail silently, when lambda function isn't found. */ fp = find_func(name); - if (fp != NULL && --fp->uf_refcount <= 0) - { - /* Only delete it when it's not being used. Otherwise it's done - * when "uf_calls" becomes zero. */ - if (fp->uf_calls == 0) - func_free(fp); - } + } + if (fp != NULL && --fp->uf_refcount <= 0) + { + /* Only delete it when it's not being used. Otherwise it's done + * when "uf_calls" becomes zero. */ + if (fp->uf_calls == 0) + func_free(fp); } } diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -759,6 +759,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2104, +/**/ 2103, /**/ 2102,