From: Duy Nguyen <pclouds@gmail.com>
Date: Wed, 04 Dec 2013 12:57:22 GMT
Subject: Re: [RFC 02/11] remote.c: add infrastructure for parsing --prune options
Message-ID: <CACsJy8DwfXPn6vJXuMyNP7W3tKe2w_mzxdw53kL=-T+0YU5wCw@mail.gmail.com>
In-Reply-To: <1386135890-13954-3-git-send-email-mhagger@alum.mit.edu>

On Wed, Dec 4, 2013 at 12:44 PM, Michael Haggerty <mhagger@alum.mit.edu> wrote:
> +void argv_push_prune_option(struct argv_array *argv,
> +                           struct prune_option *prune_option)
> +{
> +       if (prune_option->prune != -1)
> +               argv_array_pushf(argv,
> +                                prune_option->prune
> +                                ? "--prune"
> +                                : "--no-prune");
> +}

Nit. I think argv_array_push() is enough for this case?
-- 
Duy