Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > +void ref_sorting_release(struct ref_sorting *sorting) > +{ > + struct ref_sorting *next = sorting->next; > + if (next) > + ref_sorting_release(next); > + free(sorting); > +} Looks like a deep recursion that can be turned into an iteration fairly easily? I guess it does not matter as long as we won't deal with thousands of sort keys...