"Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> writes: > + const struct git_hash_algo *from = state->from; > + const struct git_hash_algo *to = state->to; > + struct strbuf *out = state->outbuf; > + const char *buffer = state->buf; > + size_t payload_size, size = state->buf_len;; The excess ';' at the end is an empty statment, hence ... > + struct object_id oid; > const char *p; > + int ret = 0; ... these three violate our "no declaration after statement" house rule.