Kacper Kornet <draenog@xxxxxxxxxxxxx> writes: > The linked list describing sort options was not correctly set up in > opt_parse_sort. In the result, contrary to the documentation. only the > last of multiple --sort options to git-for-each-ref was taken into > account. This commit fixes it. > > Signed-off-by: Kacper Kornet <draenog@xxxxxxxxxxxxx> > --- > builtin/for-each-ref.c | 4 +++- > t/t6300-for-each-ref.sh | 2 +- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c > index b01d76a..0c5294e 100644 > --- a/builtin/for-each-ref.c > +++ b/builtin/for-each-ref.c > @@ -962,7 +962,9 @@ static int opt_parse_sort(const struct option *opt, const char *arg, int unset) > if (!arg) /* should --no-sort void the list ? */ > return -1; > > - *sort_tail = s = xcalloc(1, sizeof(*s)); > + s = xcalloc(1, sizeof(*s)); > + s->next = *sort_tail; > + *sort_tail = s; This fix looks correct. -- 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