diff options
author | Bram Moolenaar <Bram@vim.org> | 2022-03-26 10:50:11 +0000 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2022-03-26 10:50:11 +0000 |
commit | c6c1ec4da53db9d292fa3dd081c20123f8261178 (patch) | |
tree | 3f6ed5205c02f8b48adf98567202bfc6792b511b | |
parent | 5e877baf87530d5c4fe4da2c5a6269cf19526c27 (diff) | |
download | vim-git-c6c1ec4da53db9d292fa3dd081c20123f8261178.tar.gz |
patch 8.2.4629: flattennew() makes a deep copy unnecessarilyv8.2.4629
Problem: flattennew() makes a deep copy unnecessarily.
Solution: Use a shallow copy. (issue #10012)
-rw-r--r-- | src/list.c | 6 | ||||
-rw-r--r-- | src/version.c | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/list.c b/src/list.c index 930904817..3354068a4 100644 --- a/src/list.c +++ b/src/list.c @@ -925,7 +925,6 @@ list_assign_range( list_flatten(list_T *list, listitem_T *first, long maxitems, long maxdepth) { listitem_T *item; - listitem_T *tofree; int done = 0; if (maxdepth == 0) @@ -955,13 +954,12 @@ list_flatten(list_T *list, listitem_T *first, long maxitems, long maxdepth) return; } clear_tv(&item->li_tv); - tofree = item; if (maxdepth > 0) list_flatten(list, item->li_prev == NULL ? list->lv_first : item->li_prev->li_next, itemlist->lv_len, maxdepth - 1); - list_free_item(list, tofree); + list_free_item(list, item); } ++done; @@ -1012,7 +1010,7 @@ flatten_common(typval_T *argvars, typval_T *rettv, int make_copy) if (make_copy) { - l = list_copy(l, TRUE, TRUE, get_copyID()); + l = list_copy(l, FALSE, TRUE, get_copyID()); rettv->vval.v_list = l; if (l == NULL) return; diff --git a/src/version.c b/src/version.c index 60b4c1d68..a68c88137 100644 --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 4629, +/**/ 4628, /**/ 4627, |