From 85ece84b1c955e2304e8519eb40ad8212a32a3ba Mon Sep 17 00:00:00 2001 From: Thomas Martitz Date: Mon, 3 Aug 2009 01:38:58 +0000 Subject: Remove various ABS() definitions with a single one using typeof (if using gcc) to avoid multiple evaluations of the input expressions. Speex still uses its own as I didn't want to change this imported code too much. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@22129 a1c6a512-1295-4272-9138-f99709370657 --- firmware/include/stdlib.h | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'firmware/include/stdlib.h') diff --git a/firmware/include/stdlib.h b/firmware/include/stdlib.h index 504b592..a287889 100644 --- a/firmware/include/stdlib.h +++ b/firmware/include/stdlib.h @@ -35,7 +35,15 @@ void *realloc(void *, size_t); void srand(unsigned int seed); int rand(void); -#define abs(x) ((x)>0?(x):-(x)) +#ifndef ABS +#if defined(__GNUC__) +#define ABS(a) ({typeof (a) ___a = (a); ___a < 0 ? -___a: ___a; }) +#else +#define ABS(a) (((a) < 0) ? -(a) : (a)) +#endif /* __GNUC__ */ +#endif + +#define abs(x) (ABS(x)) #define labs(x) abs(x) #ifdef SIMULATOR -- cgit v1.1