Re: [PATCH v2] wt-status.c: Increase code readability.

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

 



Henrik Hautakoski <henrik@xxxxxxxxxxxxx> writes:

> Some if/else statements has braces on only some branches. Be consistent and make
> all branches have braces.
>
> Signed-off-by: Henrik Hautakoski <henrik@xxxxxxxxxxxxx>

While I agree that this patch makes things consistent, and consistency may
be good, but "readable" is more subjective.

Perhaps retitle it to "Subject: wt-status.c: style fixes"?  That would be
more consistent (no pun intended) with the body of the message.

Please hold onto this patch, and make it the first one in the series of
whatever you are planning to do to wt-status.c that you mentioned.
--
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]