[clang-tidy] add parentheses to macros

Found with bugprone-macro-parentheses

Signed-off-by: Rosen Penev <rosenp@gmail.com>

5 files changed, 8 insertions, 8 deletions

diff --git a/common/debug.c b/common/debug.c
index a1c336b..6212e71 100644
--- a/common/debug.c
+++ b/common/debug.c
@@ -46,7 +46,7 @@ void internal_set_debug_level(int level)
         debug_level = level;
 }
 
-#define MAX_PRINT_LEN 16*1024
+#define MAX_PRINT_LEN (16*1024)
 
 #ifndef STRIP_DEBUG_CODE
 static void debug_print_line(const char *func, const char *file, int line, const char *buffer)
diff --git a/src/debugserver.c b/src/debugserver.c
index 7dd7f22..39c1bdc 100644
--- a/src/debugserver.c
+++ b/src/debugserver.c
@@ -239,10 +239,10 @@ static char debugserver_int2hex(int x)
         return hexchars[x];
 }
 
-#define DEBUGSERVER_HEX_ENCODE_FIRST_BYTE(byte) debugserver_int2hex((byte >> 0x4) & 0xf)
-#define DEBUGSERVER_HEX_ENCODE_SECOND_BYTE(byte) debugserver_int2hex(byte & 0xf)
-#define DEBUGSERVER_HEX_DECODE_FIRST_BYTE(byte) ((byte >> 0x4) & 0xf)
-#define DEBUGSERVER_HEX_DECODE_SECOND_BYTE(byte) (byte & 0xf)
+#define DEBUGSERVER_HEX_ENCODE_FIRST_BYTE(byte) debugserver_int2hex(((byte) >> 0x4) & 0xf)
+#define DEBUGSERVER_HEX_ENCODE_SECOND_BYTE(byte) debugserver_int2hex((byte) & 0xf)
+#define DEBUGSERVER_HEX_DECODE_FIRST_BYTE(byte) (((byte) >> 0x4) & 0xf)
+#define DEBUGSERVER_HEX_DECODE_SECOND_BYTE(byte) ((byte) & 0xf)
 
 static uint32_t debugserver_get_checksum_for_buffer(const char* buffer, uint32_t size)
 {
diff --git a/src/mobilebackup.c b/src/mobilebackup.c
index cde96b5..aa29277 100644
--- a/src/mobilebackup.c
+++ b/src/mobilebackup.c
@@ -34,7 +34,7 @@
 #define MBACKUP_VERSION_INT1 100
 #define MBACKUP_VERSION_INT2 0
 
-#define IS_FLAG_SET(x, y) ((x & y) == y)
+#define IS_FLAG_SET(x, y) (((x) & (y)) == (y))
 
 /**
  * Convert an device_link_service_error_t value to an mobilebackup_error_t value.
diff --git a/src/mobilebackup2.c b/src/mobilebackup2.c
index 6c6556d..f4fde95 100644
--- a/src/mobilebackup2.c
+++ b/src/mobilebackup2.c
@@ -33,7 +33,7 @@
 #define MBACKUP2_VERSION_INT1 400
 #define MBACKUP2_VERSION_INT2 0
 
-#define IS_FLAG_SET(x, y) ((x & y) == y)
+#define IS_FLAG_SET(x, y) (((x) & (y)) == (y))
 
 /**
  * Convert an device_link_service_error_t value to an mobilebackup2_error_t value.
diff --git a/tools/idevicebackup2.c b/tools/idevicebackup2.c
index 2f8f1fa..2170f98 100644
--- a/tools/idevicebackup2.c
+++ b/tools/idevicebackup2.c
@@ -1458,7 +1458,7 @@ static void print_usage(int argc, char **argv)
         printf("Bug Reports: <" PACKAGE_BUGREPORT ">\n");
 }
 
-#define DEVICE_VERSION(maj, min, patch) (((maj & 0xFF) << 16) | ((min & 0xFF) << 8) | (patch & 0xFF))
+#define DEVICE_VERSION(maj, min, patch) ((((maj) & 0xFF) << 16) | (((min) & 0xFF) << 8) | ((patch) & 0xFF))
 
 int main(int argc, char *argv[])
 {