From: Taylor Blau <me@ttaylorr.com>
Date: Mon, 02 Oct 2017 00:31:16 GMT
Subject: Re: [PATCH v5 0/6] Support %(trailers) arguments in for-each-ref(1)
Message-ID: <20171002003116.GA69902@D-10-157-251-166.dhcp4.washington.edu>
In-Reply-To: <20171001161725.GA66172@D-10-157-251-166.dhcp4.washington.edu>

Hi,

Attached is the fifth revision of my patch-set "Support %(trailers)
arguments in for-each-ref(1)".

It includes the following changes since v4:

  * Clarified "ref-filter.c: parse trailers arguments with %(contents)
    atom" to include reasoning for passing NULL as "" empty string in
    contents_atom_parser.

  * Changed instances "fencing" to "quoting" in "doc: use modern
    "`"-style code fencing".

  * Changed indentation in "pretty.c: delimit "%(trailers)" arguments
    with ","" to silence the output from `make style`.

  * Fix incorrect indentation in "ref-filter.c: use trailer_opts to
    format trailers".

Thanks again.


- Taylor