Function variables_is_avail_by_name is redundant so I decided to retire it because it obscures when malloc
did take place. I didn't write new functions to replace it - I just reused existing functions consistently.
variables_is_avail_by_name is retired in favor of two simpler idioms:
/* create */
var = variables_get_by_name(name); /* perform look-up */
if (var == NULL)
var = variables_new(name); /* malloc */
/* use */
var = variables_get_by_name(name); /* perform look-up */
if (var != NULL) {
/* use var */
}
Function
variables_is_avail_by_name
is redundant so I decided to retire it because it obscures when malloc did take place. I didn't write new functions to replace it - I just reused existing functions consistently.variables_is_avail_by_name
is retired in favor of two simpler idioms: