On Mon, Oct 02, 2017 at 01:51:11PM +0900, Junio C Hamano wrote:
> Taylor Blau <me@xxxxxxxxxxxx> writes:
>
> > @@ -212,9 +212,10 @@ static void contents_atom_parser(const struct ref_format *format, struct used_at
> > atom->u.contents.option = C_SIG;
> > else if (!strcmp(arg, "subject"))
> > atom->u.contents.option = C_SUB;
> > - else if (!strcmp(arg, "trailers"))
> > - atom->u.contents.option = C_TRAILERS;
> > - else if (skip_prefix(arg, "lines=", &arg)) {
> > + else if (skip_prefix(arg, "trailers", &arg)) {
> > + skip_prefix(arg, ":", &arg);
> > + trailers_atom_parser(atom, *arg ? NULL : arg);
>
> A parameter for the call is missing. I think you want 'format'
> there.
Thanks for pointing this out. I have fixed this in v6, which I am
sending out shortly. I think that this revision should be ready for
queueing.
--
- Taylor
