fix: Add brackets to avoid unexpected macro behavior

DLTcollab · Jan 3, 2020 · c33068f · c33068f
1 parent 1af1e2d
commit c33068f
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/docs/naming-convention.md b/docs/naming-convention.md
@@ -21,7 +21,7 @@ dcurl uses the **[snake case](https://en.wikipedia.org/wiki/Snake_case)** naming
 
 - Macro: Use capital letters
   ```
-  #define MIN_TRYTE_VALUE -13
+  #define MIN_TRYTE_VALUE (-13)
   #define MAX_TRYTE_VALUE 13
   ```
 
diff --git a/src/constants.h b/src/constants.h
@@ -9,7 +9,7 @@
 #ifndef CONSTANTS_H_
 #define CONSTANTS_H_
 
-#define MIN_TRYTE_VALUE -13
+#define MIN_TRYTE_VALUE (-13)
 #define MAX_TRYTE_VALUE 13
 #define SIGNATURE_SIZE 6561
 #define DEPTH 3

diff --git a/src/pow_avx.h b/src/pow_avx.h
@@ -49,7 +49,7 @@ struct pow_avx_context_s {
 #include <x86intrin.h>
 #endif
 
-#define INCR_START HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27
+#define INCR_START (HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27)
 
 #ifdef __AVX2__
 #define HBITS 0xFFFFFFFFFFFFFFFFuLL

diff --git a/src/pow_c.h b/src/pow_c.h
@@ -54,7 +54,7 @@ struct pow_c_context_s {
 
 #define HBITS 0xFFFFFFFFFFFFFFFFuLL
 #define LBITS 0x0000000000000000uLL
-#define INCR_START HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27
+#define INCR_START (HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27)
 #define LOW0 \
     0xDB6DB6DB6DB6DB6DuLL  // 0b1101101101101101101101101101101101101101101101101101101101101101L;
 #define HIGH0 \

diff --git a/src/pow_sse.h b/src/pow_sse.h
@@ -59,7 +59,7 @@ struct pow_sse_context_s {
 
 #define HBITS 0xFFFFFFFFFFFFFFFFuLL
 #define LBITS 0x0000000000000000uLL
-#define INCR_START HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27
+#define INCR_START (HASH_TRITS_LENGTH - NONCE_TRITS_LENGTH + 4 + 27)
 #define LOW00 \
     0xDB6DB6DB6DB6DB6DuLL  // 0b1101101101101101101101101101101101101101101101101101101101101101L;
 #define HIGH00 \