On Fri, 28 Jul 2023 15:04:38 -0400
Stevie Alvarez <stevie.6strings@xxxxxxxxx> wrote:
> +/**
> + * Deallocate a traceeval instance.
> + */
> +void traceeval_release(struct traceeval *teval)
> +{
> + if (teval) {
> + hist_table_release(teval);
> + type_release(teval->def_keys);
> + type_release(teval->def_vals);
> + free(teval);
> + }
> }
Nit, but it is usually cleaner to just have:
void traceeval_release(struct traceeval *teval)
{
if (!teval)
return;
hist_table_release(teval);
type_release(teval->def_keys);
type_release(teval->def_vals);
free(teval);
}
Less indents ;-)
-- Steve
|