From: Jeff King <peff@peff.net>
Date: Mon, 18 Jan 2016 19:33:38 GMT
Subject: Re: [PATCH v2 4/5] ls-remote: use parse-options api
Message-ID: <20160118193338.GC1009@sigill.intra.peff.net>
In-Reply-To: <1453136238-19448-5-git-send-email-t.gummerer@gmail.com>

On Mon, Jan 18, 2016 at 05:57:17PM +0100, Thomas Gummerer wrote:

> Currently ls-remote uses a hand rolled parser for the its command line

s/the its/its/

> arguments.  Use the parse-options api instead of the hand rolled parser
> to simplify the code and make it easier to add new arguments.  In
> addition this improves the help message.
> 
> Helped-by: Jeff King <peff@peff.net>
> Signed-off-by: Thomas Gummerer <t.gummerer@gmail.com>
> ---
>  builtin/ls-remote.c | 82 +++++++++++++++++++----------------------------------
>  1 file changed, 29 insertions(+), 53 deletions(-)

Patch itself looks good to me.

-Peff