On Wed, Jul 9, 2014 at 2:47 AM, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > On Wed, Jul 9, 2014 at 3:33 AM, Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> wrote: > > Yet, there is a space after the opening '{'. So, this is now > inconsistently formatted as: > > { foo, bar} > So, if we drop the other hunk and keep only the one below, the patch needs to be retitled with s/bracket/parenthesis/... >> @@ -298,7 +298,7 @@ int cmd_gc(int argc, const char **argv, const char *prefix) >> argv_array_pushl(&pack_refs_cmd, "pack-refs", "--all", "--prune", NULL); >> argv_array_pushl(&reflog, "reflog", "expire", "--all", NULL); >> argv_array_pushl(&repack, "repack", "-d", "-l", NULL); >> - argv_array_pushl(&prune, "prune", "--expire", NULL ); >> + argv_array_pushl(&prune, "prune", "--expire", NULL); >> argv_array_pushl(&rerere, "rerere", "gc", NULL); >> >> git_config(gc_config, NULL); >> -- >> 1.9.1.346.ga2b5940 -- 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