santiago@xxxxxxx writes: > -static int for_each_tag_name(const char **argv, each_tag_name_fn fn) > +static int for_each_tag_name(const char **argv, each_tag_name_fn fn, > + void *cb_data) > { > const char **p; > char ref[PATH_MAX]; > int had_error = 0; > unsigned char sha1[20]; > > + Why? I'll remove this while queuing. > for (p = argv; *p; p++) { > if (snprintf(ref, sizeof(ref), "refs/tags/%s", *p) > >= sizeof(ref)) {