Re: [PATCH v2] pretty: add "%aL"|"%al|%cL|%cl" option to output local-part of email addresses

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

 



Prarit Bhargava <prarit@xxxxxxxxxx> writes:

> It can't harm anything to have braces but if the preferred git coding style is
> to only use them when necessary I will remove them.

It harms readability.  Don't try to be different just for the sake
of being different, especially when you think "it can't harm
anything" aka "there is no strong reason to do so (or not to)".
Instead, help others by making sure that their eyes do not have to
get distracted for such meaningless differences from existing parts
of the system.

Thanks.



[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]

  Powered by Linux