On Tue, 3 Oct 2017 13:14:58 -0700 Brandon Williams <bmwill@xxxxxxxxxx> wrote: > +static int process_dummy_ref(void) > +{ > + struct object_id oid; > + const char *name; > + > + if (parse_oid_hex(packet_buffer, &oid, &name)) > + return 0; > + if (*name != ' ') > + return 0; > + name++; > + > + return !oidcmp(&null_oid, &oid) && !strcmp(name, "capabilities^{}"); > +} Nit: If you're planning to parse_oid_hex, you can strcmp with " capabilities^{}" directly (note the space at the start) instead of first checking for ' ' then "capabilities^{}".