Fix int wrapping for timer_expired macros and use MAX defines for consistency and clarity (#10996)

This commit is contained in:
XScorpion2 2020-11-30 04:18:19 -06:00 committed by GitHub
parent ce91dd4bf3
commit 485e4524f4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -45,8 +45,8 @@ uint16_t timer_elapsed(uint16_t last);
uint32_t timer_elapsed32(uint32_t last); uint32_t timer_elapsed32(uint32_t last);
// Utility functions to check if a future time has expired & autmatically handle time wrapping if checked / reset frequently (half of max value) // Utility functions to check if a future time has expired & autmatically handle time wrapping if checked / reset frequently (half of max value)
#define timer_expired(current, future) (((uint16_t)current - (uint16_t)future) < 0x8000) #define timer_expired(current, future) ((uint16_t)(current - future) < UINT16_MAX / 2)
#define timer_expired32(current, future) (((uint32_t)current - (uint32_t)future) < 0x80000000) #define timer_expired32(current, future) ((uint32_t)(current - future) < UINT32_MAX / 2)
#ifdef __cplusplus #ifdef __cplusplus
} }