Messages in this thread Patch in this message | | | From | Sergei Zviagintsev <> | Subject | [PATCH v2 07/10] selftests/kdbus: use parentheses in iteration macros uniformly | Date | Wed, 10 Jun 2015 00:00:05 +0300 |
| |
Enclose all arguments into parentheses in KDBUS_ITEM_FOREACH and KDBUS_FOREACH macros to stay consistent across the whole macro.
Signed-off-by: Sergei Zviagintsev <sergei@s15v.net> --- tools/testing/selftests/kdbus/kdbus-util.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/testing/selftests/kdbus/kdbus-util.h b/tools/testing/selftests/kdbus/kdbus-util.h index df5721ee8f54..d1a0f1b4d0eb 100644 --- a/tools/testing/selftests/kdbus/kdbus-util.h +++ b/tools/testing/selftests/kdbus/kdbus-util.h @@ -29,15 +29,15 @@ #define KDBUS_ITEM_NEXT(item) \ (typeof(item))((uint8_t *)(item) + KDBUS_ALIGN8((item)->size)) #define KDBUS_ITEM_FOREACH(item, head, first) \ - for (item = (head)->first; \ + for ((item) = (head)->first; \ ((uint8_t *)(item) < (uint8_t *)(head) + (head)->size) && \ ((uint8_t *)(item) >= (uint8_t *)(head)); \ - item = KDBUS_ITEM_NEXT(item)) + (item) = KDBUS_ITEM_NEXT(item)) #define KDBUS_FOREACH(iter, first, _size) \ - for (iter = (first); \ + for ((iter) = (first); \ ((uint8_t *)(iter) < (uint8_t *)(first) + (_size)) && \ ((uint8_t *)(iter) >= (uint8_t *)(first)); \ - iter = (void *)(((uint8_t *)iter) + KDBUS_ALIGN8((iter)->size))) + (iter) = (void *)((uint8_t *)(iter) + KDBUS_ALIGN8((iter)->size))) #define _KDBUS_ATTACH_BITS_SET_NR (__builtin_popcountll(_KDBUS_ATTACH_ALL)) -- 1.8.3.1
| |