commit d2c64cbd53db38ded016e1d325cf7a9c18c9bf31
author Andrew Jeffery <andrew@aj.id.au> Wed Apr 05 19:51:39 2023 +0930
committer Andrew Jeffery <andrew@aj.id.au> Thu Apr 06 03:17:55 2023 +0930
tree ba6cd2972b19d7aa953a207cc363537f446b0d63
parent 5a706073c7f826845d99bea63fb2800a89187a3e

clang-tidy: Fix readability-braces-around-statements diagnostics

For example:

```
/usr/bin/clang-tidy -checks=-*, readability-braces-around-statements -export-fixes /tmp/tmpp5g5_gym/tmpoyj8t6gj.yaml -p=build /mnt/host/andrew/src/openbmc/libpldm/src/utils.c
/mnt/host/andrew/src/openbmc/libpldm/build/../src/utils.c:88:16: error: statement should be inside braces [readability-braces-around-statements,-warnings-as-errors]
        while (size--)
                      ^
                       {
```

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

src/utils.c

diff --git a/src/utils.c b/src/utils.c
index ea25f54..482fd95 100644
--- a/src/utils.c
+++ b/src/utils.c
@@ -85,8 +85,9 @@
 {
 	const uint8_t *p = data;
 	uint32_t crc = ~0U;
-	while (size--)
+	while (size--) {
 		crc = crc32_tab[(crc ^ *p++) & 0xff] ^ (crc >> 8);
+	}
 	return crc ^ ~0U;
 }
 
@@ -94,8 +95,9 @@
 {
 	const uint8_t *p = data;
 	uint8_t crc = 0x00;
-	while (size--)
+	while (size--) {
 		crc = crc8_table[crc ^ *p++];
+	}
 	return crc;
 }
 
@@ -115,14 +117,17 @@
 	ssize_t remaining;
 	char *cursor;
 
-	if (!version || !buffer)
+	if (!version || !buffer) {
 		return -1;
+	}
 
-	if (!buffer_size)
+	if (!buffer_size) {
 		return -1;
+	}
 
-	if (buffer_size > SSIZE_MAX)
+	if (buffer_size > SSIZE_MAX) {
 		return -1;
+	}
 
 	cursor = buffer;
 	remaining = (ssize_t)buffer_size;