commit 1eeca41472ef675db0a6a4bfbe09e9aaad055d2a
author Hanno Becker <hanno.becker@arm.com> Mon Oct 15 12:01:35 2018 +0100
committer Simon Butcher <simon.butcher@arm.com> Tue Feb 19 17:59:00 2019 +0000
tree 095d907550c637a67d68271e764a7069430e8af2
parent 9c99dc862cd5cf9cdc62cbfcd200063c83295397

Improve macro hygiene

This commit improves hygiene and formatting of macro definitions
throughout the library. Specifically:
- It adds brackets around parameters to avoid unintended
  interpretation of arguments, e.g. due to operator precedence.
- It adds uses of the `do { ... } while( 0 )` idiom for macros that
  can be used as commands.

include/mbedtls/padlock.h

diff --git a/include/mbedtls/padlock.h b/include/mbedtls/padlock.h
index f05b72b..721a5d4 100644
--- a/include/mbedtls/padlock.h
+++ b/include/mbedtls/padlock.h
@@ -59,7 +59,7 @@
 #define MBEDTLS_PADLOCK_PHE 0x0C00
 #define MBEDTLS_PADLOCK_PMM 0x3000
 
-#define MBEDTLS_PADLOCK_ALIGN16(x) (uint32_t *) (16 + ((int32_t) x & ~15))
+#define MBEDTLS_PADLOCK_ALIGN16(x) (uint32_t *) (16 + ((int32_t) (x) & ~15))
 
 #ifdef __cplusplus
 extern "C" {