Phillip Wood <phillip.wood123@xxxxxxxxx> writes: >> +static void release_revisions_commit_list(struct rev_info *revs) >> +{ >> + if (revs->commits) { > > This guard copied for the original is not needed, free_commit_list() > will handle NULL just fine. Thanks for a careful reading.