commit 2834c5b149151347309948034da4d8bb86ca5fde
author Andrew Jeffery <andrew@aj.id.au> Wed Apr 19 12:47:09 2023 +0930
committer Andrew Jeffery <andrew@aj.id.au> Fri Apr 21 10:49:12 2023 +0930
tree 4e1cf152d5195e0fb5b9d334a09f74e8c41f9cca
parent 4ee419df1d510a29b49cdf6f52168a893e2fa47b

obmc-console: Fix readability-braces-around-statements

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I88d2bdcb15106298d83a1cf7176e069092820f1d

config.c

diff --git a/config.c b/config.c
index 6fc840c..ce2ed47 100644
--- a/config.c
+++ b/config.c
@@ -47,9 +47,11 @@
 {
 	struct config_item *item;
 
-	for (item = config->items; item; item = item->next)
-		if (!strcasecmp(item->name, name))
+	for (item = config->items; item; item = item->next) {
+		if (!strcasecmp(item->name, name)) {
 			return item->value;
+		}
+	}
 
 	return NULL;
 }
@@ -65,12 +67,14 @@
 		int rc;
 
 		/* trim leading space */
-		for (; *line == ' ' || *line == '\t'; line++)
+		for (; *line == ' ' || *line == '\t'; line++) {
 			;
+		}
 
 		/* skip comments */
-		if (*line == '#')
+		if (*line == '#') {
 			continue;
+		}
 
 		name = malloc(strlen(line));
 		value = malloc(strlen(line));
@@ -139,8 +143,9 @@
 	struct config *config;
 	int fd;
 
-	if (!filename)
+	if (!filename) {
 		filename = config_default_filename;
+	}
 
 	fd = open(filename, O_RDONLY);
 	if (fd < 0) {
@@ -271,16 +276,19 @@
 	char *suffix;
 	size_t i;
 
-	if (!size_str)
+	if (!size_str) {
 		return -1;
+	}
 
 	logsize = strtoul(size_str, &suffix, 0);
-	if (logsize == 0 || logsize >= UINT32_MAX || suffix == size_str)
+	if (logsize == 0 || logsize >= UINT32_MAX || suffix == size_str) {
 		return -1;
+	}
 
 	/* Ignore spaces between number and suffix */
-	while (*suffix && isspace(*suffix))
+	while (*suffix && isspace(*suffix)) {
 		suffix++;
+	}
 
 	for (i = 0; i < num_suffixes; i++) {
 		if (*suffix == suffixes[i].unit) {
@@ -288,8 +296,9 @@
 			 * If logsize overflows, probably something was wrong.
 			 * Return instead of clamping to an arbitrary value.
 			 */
-			if (logsize > (UINT32_MAX >> suffixes[i].shiftwidth))
+			if (logsize > (UINT32_MAX >> suffixes[i].shiftwidth)) {
 				return -1;
+			}
 
 			logsize <<= suffixes[i].shiftwidth;
 			suffix++;
@@ -298,8 +307,9 @@
 	}
 
 	/* Allow suffix like 'kB' */
-	while (*suffix && (tolower(*suffix) == 'b' || isspace(*suffix)))
+	while (*suffix && (tolower(*suffix) == 'b' || isspace(*suffix))) {
 		suffix++;
+	}
 
 	if (*suffix) {
 		warn("Invalid suffix!");