On Tue, 2016-01-12 at 16:19 -0800, Junio C Hamano wrote: > David Turner <dturner@xxxxxxxxxxxxxxxx> writes: > > > +static int dereference_symrefs(struct ref_transaction > > *transaction, > > + struct strbuf *err) > > +{ > > + int i; > > + int nr = transaction->nr; > > + > > + for (i = 0; i < nr; i++) { > > + struct ref_update *update = transaction > > ->updates[i]; > > + const char *resolved; > > + unsigned char sha1[20]; > > + int resolve_flags = 0; > > + int mustexist = (update->old_sha1 && > > + !is_null_sha1(update->old_sha1)); > > + int deleting = (update->flags & REF_HAVE_NEW) && > > + is_null_sha1(update->new_sha1); > > + struct ref_update *new_update; > > This variable is never used (at least at the end of the series after > applying up to 21/21). Fixed, thanks. -- 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