# HG changeset patch # User vimboss # Date 1126128147 0 # Node ID f607f15ab091b6a51b0546db6a6d42adfa599c3d # Parent 32cf0d2e14b4b0ac22e46cc19d56eee49431c6c2 updated for version 7.0143 diff --git a/src/eval.c b/src/eval.c --- a/src/eval.c +++ b/src/eval.c @@ -2831,6 +2831,8 @@ eval_for_line(arg, errp, nextcmdp, skip) } else { + /* No need to increment the refcount, it's already set for the + * list being used in "tv". */ fi->fi_list = l; list_add_watch(l, &fi->fi_lw); fi->fi_lw.lw_item = l->lv_first; @@ -5861,14 +5863,16 @@ garbage_collect() /* * 3. Go through the list of lists and free items without the copyID. + * But don't free a list that has a watcher (used in a for loop), these + * are not referenced anywhere. */ for (ll = first_list; ll != NULL; ) - if (ll->lv_copyID != copyID) + if (ll->lv_copyID != copyID && ll->lv_watch == NULL) { list_free(ll); did_free = TRUE; - /* restart, next dict may also have been freed */ + /* restart, next list may also have been freed */ ll = first_list; } else