From: Joseph Sutton <josephsutton@catalyst.net.nz>
Date: Thu, 30 Nov 2023 02:11:10 +0000 (+1300)
Subject: lib:util: Don’t unnecessarily parenthesize macro arguments
X-Git-Tag: talloc-2.4.2~228
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=7d88280baaa8289addcc50fa927a6ac856628871;p=thirdparty%2Fsamba.git

lib:util: Don’t unnecessarily parenthesize macro arguments

If we’re just passing a parameter to another macro which we know
correctly parenthesizes its arguments, then we don’t need to
parenthesize the parameter ourselves.

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
---

diff --git a/lib/util/bytearray.h b/lib/util/bytearray.h
index f39340a5ee1..ec5b7a01c63 100644
--- a/lib/util/bytearray.h
+++ b/lib/util/bytearray.h
@@ -59,17 +59,17 @@
     PUSH_LE_U8(data, pos, val)
 
 #define PUSH_LE_U16(data, pos, val) \
-    (PUSH_LE_U8((data), (pos), (uint8_t)((uint16_t)(val) & 0xff)), PUSH_LE_U8((data), (pos) + 1, (uint8_t)((uint16_t)(val) >> 8)))
+    (PUSH_LE_U8(data, pos, (uint8_t)((uint16_t)(val) & 0xff)), PUSH_LE_U8(data, (pos) + 1, (uint8_t)((uint16_t)(val) >> 8)))
 #define PUSH_LE_I16(data, pos, val) \
     PUSH_LE_U16(data, pos, val)
 
 #define PUSH_LE_U32(data, pos, val) \
-    (PUSH_LE_U16((data), (pos), (uint16_t)((uint32_t)(val) & 0xffff)), PUSH_LE_U16((data), (pos) + 2, (uint16_t)((uint32_t)(val) >> 16)))
+    (PUSH_LE_U16(data, pos, (uint16_t)((uint32_t)(val) & 0xffff)), PUSH_LE_U16(data, (pos) + 2, (uint16_t)((uint32_t)(val) >> 16)))
 #define PUSH_LE_I32(data, pos, val) \
     PUSH_LE_U32(data, pos, val)
 
 #define PUSH_LE_U64(data, pos, val) \
-    (PUSH_LE_U32((data), (pos), (uint32_t)((uint64_t)(val) & 0xffffffff)), PUSH_LE_U32((data), (pos) + 4, (uint32_t)((uint64_t)(val) >> 32)))
+    (PUSH_LE_U32(data, pos, (uint32_t)((uint64_t)(val) & 0xffffffff)), PUSH_LE_U32(data, (pos) + 4, (uint32_t)((uint64_t)(val) >> 32)))
 #define PUSH_LE_I64(data, pos, val) \
     PUSH_LE_U64(data, pos, val)
 
@@ -107,17 +107,17 @@
     PUSH_BE_U8(data, pos, val)
 
 #define PUSH_BE_U16(data, pos, val) \
-    (PUSH_BE_U8((data), (pos), (uint8_t)(((uint16_t)(val)) >> 8)), PUSH_BE_U8((data), (pos) + 1, (uint8_t)((uint16_t)(val) & 0xff)))
+    (PUSH_BE_U8(data, pos, (uint8_t)(((uint16_t)(val)) >> 8)), PUSH_BE_U8(data, (pos) + 1, (uint8_t)((uint16_t)(val) & 0xff)))
 #define PUSH_BE_I16(data, pos, val) \
     PUSH_BE_U16(data, pos, val)
 
 #define PUSH_BE_U32(data, pos, val) \
-    (PUSH_BE_U16((data), (pos), (uint16_t)(((uint32_t)(val)) >> 16)), PUSH_BE_U16((data), (pos) + 2, (uint16_t)((uint32_t)(val) & 0xffff)))
+    (PUSH_BE_U16(data, pos, (uint16_t)(((uint32_t)(val)) >> 16)), PUSH_BE_U16(data, (pos) + 2, (uint16_t)((uint32_t)(val) & 0xffff)))
 #define PUSH_BE_I32(data, pos, val) \
     PUSH_BE_U32(data, pos, val)
 
 #define PUSH_BE_U64(data, pos, val) \
-    (PUSH_BE_U32((data), (pos), (uint32_t)(((uint64_t)(val)) >> 32)), PUSH_BE_U32((data), (pos) + 4, (uint32_t)((uint64_t)(val) & 0xffffffff)))
+    (PUSH_BE_U32(data, pos, (uint32_t)(((uint64_t)(val)) >> 32)), PUSH_BE_U32(data, (pos) + 4, (uint32_t)((uint64_t)(val) & 0xffffffff)))
 #define PUSH_BE_I64(data, pos, val) \
     PUSH_BE_U64(data, pos, val)