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.

library/x509_crt.c

diff --git a/library/x509_crt.c b/library/x509_crt.c
index 1b1f0a7..c617c08 100644
--- a/library/x509_crt.c
+++ b/library/x509_crt.c
@@ -1444,7 +1444,7 @@
     }
 
 #define CERT_TYPE(type,name)                    \
-    if( ns_cert_type & type )                   \
+    if( ns_cert_type & (type) )                 \
         PRINT_ITEM( name );
 
 static int x509_info_cert_type( char **buf, size_t *size,
@@ -1471,7 +1471,7 @@
 }
 
 #define KEY_USAGE(code,name)    \
-    if( key_usage & code )      \
+    if( key_usage & (code) )    \
         PRINT_ITEM( name );
 
 static int x509_info_key_usage( char **buf, size_t *size,