Christian Couder <christian.couder@xxxxxxxxx> writes: >> /* Simple per-ref filtering */ >> - if (!filter->kind) >> + if (!filter->kind) { >> die("filter_refs: invalid type"); >> - else { >> + } else { > > Nit: the `else` could be removed altogether here, but maybe that > should be done in a preparatory patch. Good eyes. Thanks for carefully reading it over.