From: Junio C Hamano Date: Tue, 26 Apr 2011 17:13:20 GMT Subject: Re: [PATCH v2] wt-status.c: Increase code readability. Message-ID: <7vei4pey1b.fsf@alter.siamese.dyndns.org> In-Reply-To: <1303817198-23347-1-git-send-email-henrik@fiktivkod.org> Henrik Hautakoski writes: > Some if/else statements has braces on only some branches. Be consistent and make > all branches have braces. > > Signed-off-by: Henrik Hautakoski 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.