Thread (1 message) 1 message, 1 author, 2012-02-03
DORMANTno replies

[PATCH 03/10] Trivial style fixup

From: David Gibson <hidden>
Date: 2012-02-03 05:12:02
Subsystem: the rest · Maintainer: Linus Torvalds

Having braces on an if branch but not the else branch, or vice
versa is ugly and can trick you when reading the code.

Signed-off-by: David Gibson <redacted>
---
 fdtget.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/fdtget.c b/fdtget.c
index 48ab615..2c384b6 100644
--- a/fdtget.c
+++ b/fdtget.c
@@ -77,9 +77,9 @@ static int show_data(struct display_info *disp, const char *data, int len)
 		return 0;
 	}
 	size = disp->size;
-	if (size == -1)
+	if (size == -1) {
 		size = (len % 4) == 0 ? 4 : 1;
-	else if (len % size) {
+	} else if (len % size) {
 		fprintf(stderr, "Property length must be a multiple of "
 				"selected data size\n");
 		return -1;
-- 
1.7.8.3
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help