Re: [PATCH v6 28/32] gc: style change -- no SP before closing bracket

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]