]> Kevux Git Server - fll/commitdiff
Update: Add additional parenthesis around status in F_status check macros.
authorKevin Day <thekevinday@gmail.com>
Sun, 14 Jan 2024 00:01:29 +0000 (18:01 -0600)
committerKevin Day <thekevinday@gmail.com>
Sun, 14 Jan 2024 00:01:29 +0000 (18:01 -0600)
This is an extra safety check just in case.

level_0/f_status/c/status.h

index 9bbd2583d37976dab1c408660b9c3b2f575f6254..726e9674df6a768d791bae12025857cf19730b7c 100644 (file)
@@ -93,22 +93,22 @@ extern "C" {
   #define F_status_mask_fine 0x3fff   // 0011 1111 1111 1111
   #define F_status_mask_code 0xc000   // 1100 0000 0000 0000
 
-  #define F_status_is_error(status)   (status & F_status_bit_error)
-  #define F_status_is_fine(status)    ((status & F_status_mask_code) == 0)
-  #define F_status_is_problem(status) (status & F_status_mask_code)
-  #define F_status_is_signal(status)  ((status & F_status_bit_signal) == F_status_bit_signal)
-  #define F_status_is_warning(status) (status & F_status_bit_warning)
+  #define F_status_is_error(status)   ((status) & F_status_bit_error)
+  #define F_status_is_fine(status)    (((status) & F_status_mask_code) == 0)
+  #define F_status_is_problem(status) ((status) & F_status_mask_code)
+  #define F_status_is_signal(status)  (((status) & F_status_bit_signal) == F_status_bit_signal)
+  #define F_status_is_warning(status) ((status) & F_status_bit_warning)
 
-  #define F_status_is_error_not(status)   ((status & F_status_bit_error) == 0)
-  #define F_status_is_fine_not(status)    (status & F_status_mask_code)
-  #define F_status_is_problem_not(status) ((status & F_status_mask_code) == 0)
-  #define F_status_is_signal_not(status)  ((status & F_status_bit_signal) != F_status_bit_signal)
-  #define F_status_is_warning_not(status) ((status & F_status_bit_warning) == 0)
+  #define F_status_is_error_not(status)   (((status) & F_status_bit_error) == 0)
+  #define F_status_is_fine_not(status)    ((status) & F_status_mask_code)
+  #define F_status_is_problem_not(status) (((status) & F_status_mask_code) == 0)
+  #define F_status_is_signal_not(status)  (((status) & F_status_bit_signal) != F_status_bit_signal)
+  #define F_status_is_warning_not(status) (((status) & F_status_bit_warning) == 0)
 
-  #define F_status_set_error(status)   (status | F_status_bit_error)
-  #define F_status_set_fine(status)    (status & F_status_mask_fine)
-  #define F_status_set_signal(status)  (status | F_status_bit_signal)
-  #define F_status_set_warning(status) (status | F_status_bit_warning)
+  #define F_status_set_error(status)   ((status) | F_status_bit_error)
+  #define F_status_set_fine(status)    ((status) & F_status_mask_fine)
+  #define F_status_set_signal(status)  ((status) | F_status_bit_signal)
+  #define F_status_set_warning(status) ((status) | F_status_bit_warning)
 
   #define F_status_size_max_d           0x3fff
   #define F_status_size_max_with_bits_d 0xffff