Junio C Hamano <gitster@xxxxxxxxx> writes: >> +static struct trailer_item *create_trailer_item(const char *string) >> +{ >> + struct strbuf tok = STRBUF_INIT; >> + struct strbuf val = STRBUF_INIT; >> + struct trailer_item *new; >> + >> + parse_trailer(&tok, &val, string); >> + >> + int tok_alnum_len = alnum_len(tok.buf, tok.len); > > decl-after-stmt. > >> + >> + /* Lookup if the token matches something in the config */ >> + struct trailer_item *item; ditto. >> + for (item = first_conf_item; item; item = item->next) >> + { Style. I wonder if Cc list is being a bit too wide for this series, by the way. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html