I'm using 't' as the name for a pointer to a variable.
Using 'v' makes more sense. (t was short for 'temp').
Signed-off-by: NeilBrown <neil@brown.name>
context.varlist = b->next;
free(b);
while (v) {
- struct variable *t = v;
+ struct variable *next = v->previous;
- v = t->previous;
- if (t->global) {
- free_value(t->type, var_value(&context, t));
- if (t->depth == 0)
- free_exec(t->where_decl);
+ if (v->global) {
+ free_value(v->type, var_value(&context, v));
+ if (v->depth == 0)
+ // This is a global constant
+ free_exec(v->where_decl);
}
- free(t);
+ free(v);
+ v = next;
}
}