]> granicus.if.org Git - vim/commitdiff
patch 8.2.0473: variables declared in an outer scope v8.2.0473
authorBram Moolenaar <Bram@vim.org>
Sun, 29 Mar 2020 13:12:15 +0000 (15:12 +0200)
committerBram Moolenaar <Bram@vim.org>
Sun, 29 Mar 2020 13:12:15 +0000 (15:12 +0200)
Problem:    Variables declared in an outer scope.
Solution:   Decleare variables only in the scope where they are used.

src/evalvars.c
src/version.c

index 40a22f69075747e08cd91239c50057d438b24df9..0ffff568638e4eaefdd4586a60aec582cc018114 100644 (file)
@@ -3461,7 +3461,6 @@ f_getwinvar(typval_T *argvars, typval_T *rettv)
 f_getbufvar(typval_T *argvars, typval_T *rettv)
 {
     buf_T      *buf;
-    buf_T      *save_curbuf;
     char_u     *varname;
     dictitem_T *v;
     int                done = FALSE;
@@ -3476,12 +3475,13 @@ f_getbufvar(typval_T *argvars, typval_T *rettv)
 
     if (buf != NULL && varname != NULL)
     {
-       // set curbuf to be our buf, temporarily
-       save_curbuf = curbuf;
-       curbuf = buf;
-
        if (*varname == '&')
        {
+           buf_T       *save_curbuf = curbuf;
+
+           // set curbuf to be our buf, temporarily
+           curbuf = buf;
+
            if (varname[1] == NUL)
            {
                // get all buffer-local options in a dict
@@ -3496,22 +3496,21 @@ f_getbufvar(typval_T *argvars, typval_T *rettv)
            else if (get_option_tv(&varname, rettv, TRUE) == OK)
                // buffer-local-option
                done = TRUE;
+
+           // restore previous notion of curbuf
+           curbuf = save_curbuf;
        }
        else
        {
            // Look up the variable.
            // Let getbufvar({nr}, "") return the "b:" dictionary.
-           v = find_var_in_ht(&curbuf->b_vars->dv_hashtab,
-                                                        'b', varname, FALSE);
+           v = find_var_in_ht(&buf->b_vars->dv_hashtab, 'b', varname, FALSE);
            if (v != NULL)
            {
                copy_tv(&v->di_tv, rettv);
                done = TRUE;
            }
        }
-
-       // restore previous notion of curbuf
-       curbuf = save_curbuf;
     }
 
     if (!done && argvars[2].v_type != VAR_UNKNOWN)
@@ -3618,11 +3617,11 @@ f_setbufvar(typval_T *argvars, typval_T *rettv UNUSED)
        }
        else
        {
-           buf_T *save_curbuf = curbuf;
-
            bufvarname = alloc(STRLEN(varname) + 3);
            if (bufvarname != NULL)
            {
+               buf_T *save_curbuf = curbuf;
+
                curbuf = buf;
                STRCPY(bufvarname, "b:");
                STRCPY(bufvarname + 2, varname);
index 59e601421defb54e504f7aa69044bbc0a01ba2e5..3282033587068ec9c53bf212e9941d5e862adc45 100644 (file)
@@ -738,6 +738,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    473,
 /**/
     472,
 /**/