On 3/9/2022 8:16 AM, Ævar Arnfjörð Bjarmason wrote: > +static int cmd_log_deinit(int ret, struct rev_info *rev) > +{ > + release_revisions(rev); > + return ret; > +} This pattern of passing a return value through the helper function is a clever way to get around adding "int ret = ...; release(); return ret;" lines. Thanks, -Stolee