On Thu, Jul 13, 2017 at 8:01 AM, Jeff King <peff@xxxxxxxx> wrote: > builtin/branch.c | 14 +++++++------- > builtin/for-each-ref.c | 22 ++++++++++++---------- > builtin/tag.c | 30 ++++++++++++++++-------------- > builtin/verify-tag.c | 12 ++++++------ > ref-filter.c | 22 ++++++++++++---------- > ref-filter.h | 22 +++++++++++++++++----- > 6 files changed, 70 insertions(+), 52 deletions(-) The patch looks good to me. So some off-topic comments: I reviewed this patch from bottom up, i.e. I started looking at ref-filter.h, then ref-filter.c and then the rest. If only you had formatted the patches with an orderfile. ;)