diff options
author | Arnold D. Robbins <arnold@skeeve.com> | 2013-09-22 16:57:11 +0300 |
---|---|---|
committer | Arnold D. Robbins <arnold@skeeve.com> | 2013-09-22 16:57:11 +0300 |
commit | da83b6857bf0a67b15fc75d31a0b6802ac9baffe (patch) | |
tree | 399e9f1ec800b5405f9b494060481a8420480caf /array.c | |
parent | e149eb882355f427d43928324145c971a0562c5e (diff) | |
parent | 8aa14c5f3cf78f90b589785a9ffe5f7f02050b37 (diff) | |
download | egawk-da83b6857bf0a67b15fc75d31a0b6802ac9baffe.tar.gz egawk-da83b6857bf0a67b15fc75d31a0b6802ac9baffe.tar.bz2 egawk-da83b6857bf0a67b15fc75d31a0b6802ac9baffe.zip |
Merge branch 'master' into comment
Diffstat (limited to 'array.c')
-rw-r--r-- | array.c | 18 |
1 files changed, 7 insertions, 11 deletions
@@ -113,21 +113,14 @@ null_array(NODE *symbol) symbol->table_size = symbol->array_size = 0; symbol->array_capacity = 0; symbol->flags = 0; - /* - * 5/2013: This used to be - * - * assert(symbol->xarray == NULL); - * - * But that seems to cause problems for no good reason - * that I can see. I believe it to be an artifact of the - * union getting in the way. - */ - symbol->xarray = NULL; + + assert(symbol->xarray == NULL); + /* vname, parent_array not (re)initialized */ } -/* null_lookup: assign type to an empty array. */ +/* null_lookup --- assign type to an empty array. */ static NODE ** null_lookup(NODE *symbol, NODE *subs) @@ -349,6 +342,7 @@ force_array(NODE *symbol, bool canfatal) switch (symbol->type) { case Node_var_new: + symbol->xarray = NULL; /* make sure union is as it should be */ null_array(symbol); symbol->parent_array = NULL; /* main array has no parent */ /* fall through */ @@ -854,6 +848,8 @@ asort_actual(int nargs, sort_context_t ctxt) /* source array is empty */ if (dest != NULL && dest != array) assoc_clear(dest); + if (list != NULL) + efree(list); return make_number((AWKNUM) 0); } |