From 56caa06677f058d2190d0e91bcb54ff6f6a24518 Mon Sep 17 00:00:00 2001 From: Olaf Bergmann Date: Fri, 25 May 2018 13:45:31 +0200 Subject: [PATCH] test_session.c: fix warning for taken absolute value of unsigned int This change replaces the call of abs(v - ref) to avoid a compiler warning for -Wabsolute-value: test_session.c:95:11: warning: taking the absolute value of unsigned type 'unsigned int' has no effect [-Wabsolute-value] return (abs(v - ref) / 100.0) <= FP_ERR_THRESHOLD; --- tests/test_session.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test_session.c b/tests/test_session.c index 703900cd..2c81667c 100644 --- a/tests/test_session.c +++ b/tests/test_session.c @@ -92,7 +92,8 @@ timeout(const coap_fixed_point_t ack_timeout, */ COAP_STATIC_INLINE int good_enough(unsigned int v, unsigned int ref) { - return (abs(v - ref) / 100.0) <= FP_ERR_THRESHOLD; +#define delta(a,b) (((a) < (b)) ? ((b) - (a)) : ((a) - (b))) + return (delta(v,ref) / 100.0) <= FP_ERR_THRESHOLD; } static void