]> git.ipfire.org Git - thirdparty/vim.git/commitdiff
patch 8.2.2075: error for const argument to mapnew() v8.2.2075
authorBram Moolenaar <Bram@vim.org>
Tue, 1 Dec 2020 20:08:05 +0000 (21:08 +0100)
committerBram Moolenaar <Bram@vim.org>
Tue, 1 Dec 2020 20:08:05 +0000 (21:08 +0100)
Problem:    Error for const argument to mapnew().
Solution:   Don't give an error. (closes #7400)

src/list.c
src/testdir/test_filter_map.vim
src/version.c

index c364fa72fda590d0dee21501d7a2c817e93faf63..6285e01d502c4748eb647b3aeb107eacc746da1e 100644 (file)
@@ -2065,7 +2065,7 @@ filter_map(typval_T *argvars, typval_T *rettv, filtermap_T filtermap)
 
                    --todo;
                    di = HI2DI(hi);
-                   if (filtermap != FILTERMAP_FILTER
+                   if (filtermap == FILTERMAP_MAP
                                         && (value_check_lock(di->di_tv.v_lock,
                                                           arg_errmsg, TRUE)
                                || var_check_ro(di->di_flags,
@@ -2225,12 +2225,12 @@ filter_map(typval_T *argvars, typval_T *rettv, filtermap_T filtermap)
            }
            else
            {
-               // Materialized list from range(): loop over the items
+               // Materialized list: loop over the items
                for (li = l->lv_first; li != NULL; li = nli)
                {
                    typval_T newtv;
 
-                   if (filtermap != FILTERMAP_FILTER && value_check_lock(
+                   if (filtermap == FILTERMAP_MAP && value_check_lock(
                                           li->li_tv.v_lock, arg_errmsg, TRUE))
                        break;
                    nli = li->li_next;
index e88f755b9d2531b837a1f4a186cbb084d6145947..a9ff35e1b59af8f0829fe0fd459796d7f5ed7c89 100644 (file)
@@ -123,6 +123,9 @@ func Test_mapnew_dict()
   let dout = mapnew(din, {k, v -> string(v)})
   call assert_equal(#{one: 1, two: 2}, din)
   call assert_equal(#{one: '1', two: '2'}, dout)
+
+  const dconst = #{one: 1, two: 2, three: 3}
+  call assert_equal(#{one: 2, two: 3, three: 4}, mapnew(dconst, {_, v -> v + 1}))
 endfunc
 
 func Test_mapnew_list()
@@ -130,6 +133,9 @@ func Test_mapnew_list()
   let lout = mapnew(lin, {k, v -> string(v)})
   call assert_equal([1, 2, 3], lin)
   call assert_equal(['1', '2', '3'], lout)
+
+  const lconst = [1, 2, 3]
+  call assert_equal([2, 3, 4], mapnew(lconst, {_, v -> v + 1}))
 endfunc
 
 func Test_mapnew_blob()
index 469b92d3179179d49f544554aa567bce11757c91..bbbf9f23e62cbd73c24c785afce828eee0fee5e0 100644 (file)
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2075,
 /**/
     2074,
 /**/