On 12/04/2013 01:57 PM, Duy Nguyen wrote: > On Wed, Dec 4, 2013 at 12:44 PM, Michael Haggerty <mhagger@xxxxxxxxxxxx> 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? Yes, you're right. Thanks for noticing. If people want this patch series I will fix the problem in the next round. Michael -- Michael Haggerty mhagger@xxxxxxxxxxxx http://softwareswirl.blogspot.com/ -- 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