From 3d1e99d01bc0142397cd0dd8be41d001f0f66131 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 28 Jun 2018 13:19:42 -0400 Subject: [PATCH] Move MIN and MAX into lib/intmath/cmp.h --- src/common/compat.h | 14 +------------- src/lib/intmath/cmp.h | 13 +++++++++++++ 2 files changed, 14 insertions(+), 13 deletions(-) diff --git a/src/common/compat.h b/src/common/compat.h index 43e98d906c..ec4df1cc55 100644 --- a/src/common/compat.h +++ b/src/common/compat.h @@ -60,6 +60,7 @@ #include "lib/fs/mmap.h" #include "lib/fs/userdb.h" #include "lib/wallclock/timeval.h" +#include "lib/intmath/cmp.h" #include #include @@ -92,19 +93,6 @@ typedef unsigned long rlim_t; int set_max_file_descriptors(rlim_t limit, int *max); MOCK_DECL(int, get_total_system_memory, (size_t *mem_out)); -/** Macros for MIN/MAX. Never use these when the arguments could have - * side-effects. - * {With GCC extensions we could probably define a safer MIN/MAX. But - * depending on that safety would be dangerous, since not every platform - * has it.} - **/ -#ifndef MAX -#define MAX(a,b) ( ((a)<(b)) ? (b) : (a) ) -#endif -#ifndef MIN -#define MIN(a,b) ( ((a)>(b)) ? (b) : (a) ) -#endif - ssize_t tor_getpass(const char *prompt, char *output, size_t buflen); /* This needs some of the declarations above so we include it here. */ diff --git a/src/lib/intmath/cmp.h b/src/lib/intmath/cmp.h index 90ea3ca079..627e5d18b4 100644 --- a/src/lib/intmath/cmp.h +++ b/src/lib/intmath/cmp.h @@ -6,6 +6,19 @@ #ifndef TOR_INTMATH_CMP_H #define TOR_INTMATH_CMP_H +/** Macros for MIN/MAX. Never use these when the arguments could have + * side-effects. + * {With GCC extensions we could probably define a safer MIN/MAX. But + * depending on that safety would be dangerous, since not every platform + * has it.} + **/ +#ifndef MAX +#define MAX(a,b) ( ((a)<(b)) ? (b) : (a) ) +#endif +#ifndef MIN +#define MIN(a,b) ( ((a)>(b)) ? (b) : (a) ) +#endif + /* Return v if it's between min and max. Otherwise * return min if v is smaller than min, or max if * b is larger than max.