summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar Rosen Penev2020-12-23 17:31:34 -0800
committerGravatar Nikias Bassen2022-04-22 13:22:38 +0200
commit95316d81633120244d53b85303447beb1a917f74 (patch)
treed836d7b096a86bc64b1e64c09f6c4433b0df67cb
parent78040882a30666f29985bc6589be03cbc7d7cfbe (diff)
downloadlibimobiledevice-95316d81633120244d53b85303447beb1a917f74.tar.gz
libimobiledevice-95316d81633120244d53b85303447beb1a917f74.tar.bz2
[clang-tidy] add parentheses to macros
Found with bugprone-macro-parentheses Signed-off-by: Rosen Penev <rosenp@gmail.com>
-rw-r--r--common/debug.c2
-rw-r--r--src/debugserver.c8
-rw-r--r--src/mobilebackup.c2
-rw-r--r--src/mobilebackup2.c2
-rw-r--r--tools/idevicebackup2.c2
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[])
{