| commit | b1133e9565e118ac23279f8ad01bbc8c161f40c4 | [log] [tgz] |
|---|---|---|
| author | David Benjamin <davidben@google.com> | Tue Oct 18 13:05:01 2016 -0400 |
| committer | Adam Langley <agl@google.com> | Tue Oct 18 18:28:23 2016 +0000 |
| tree | b27ca476dda470e9bff25334d1ece217f4616c02 | |
| parent | 840445d406c36773fd85c5cad938a79ee2499cd7 [diff] [blame] |
Fix up macros. Macros need a healthy dose of parentheses to avoid expression-level misparses. Most of this comes from the clang-tidy CL here: https://android-review.googlesource.com/c/235696/ Also switch most of the macros to use do { ... } while (0) to avoid all the excessive comma operators and statement-level misparses. Change-Id: I4c2ee51e347d2aa8c74a2d82de63838b03bbb0f9 Reviewed-on: https://boringssl-review.googlesource.com/11660 Commit-Queue: David Benjamin <davidben@google.com> Reviewed-by: Adam Langley <agl@google.com>
diff --git a/include/openssl/x509_vfy.h b/include/openssl/x509_vfy.h index 0a45aad..1182bc2 100644 --- a/include/openssl/x509_vfy.h +++ b/include/openssl/x509_vfy.h
@@ -110,7 +110,7 @@ */ /* The following are legacy constants that should not be used. */ -#define X509_LU_RETRY -1 +#define X509_LU_RETRY (-1) #define X509_LU_FAIL 0 #define X509_LU_X509 1