diff options
| author | Nathan Scott <nathans@redhat.com> | 2020-09-09 16:56:04 +1000 |
|---|---|---|
| committer | Nathan Scott <nathans@redhat.com> | 2020-09-09 16:56:04 +1000 |
| commit | c5808c56db166528ae7e74cedb51cc466f973b9f (patch) | |
| tree | b0364c031595aacc8db222885bebb8390983429e /freebsd | |
| parent | 8ec5d4a3a07fe9a3c556a4abeac6645a7bb2235e (diff) | |
Consolidate repeated macro definitions into one header
The MIN, MAX, CLAMP, MINIMUM, and MAXIMUM macros appear
throughout the codebase with many re-definitions. Make
a single copy of each in a common header file, and use
the BSD variants of MINIMUM/MAXIMUM due to conflicts in
the system <sys/param.h> headers.
Diffstat (limited to 'freebsd')
| -rw-r--r-- | freebsd/FreeBSDProcessList.c | 2 | ||||
| -rw-r--r-- | freebsd/Platform.c | 4 | ||||
| -rw-r--r-- | freebsd/Platform.h | 4 |
3 files changed, 1 insertions, 9 deletions
diff --git a/freebsd/FreeBSDProcessList.c b/freebsd/FreeBSDProcessList.c index 344c3314..a4deaad9 100644 --- a/freebsd/FreeBSDProcessList.c +++ b/freebsd/FreeBSDProcessList.c @@ -111,7 +111,7 @@ ProcessList* ProcessList_new(UsersTable* usersTable, Hashtable* pidWhiteList, ui sysctl(MIB_kern_cp_times, 2, fpl->cp_times_o, &len, NULL, 0); } - pl->cpuCount = MAX(cpus, 1); + pl->cpuCount = MAXIMUM(cpus, 1); if (cpus == 1 ) { fpl->cpus = xRealloc(fpl->cpus, sizeof(CPUData)); diff --git a/freebsd/Platform.c b/freebsd/Platform.c index 58a9bf1b..c51b37cb 100644 --- a/freebsd/Platform.c +++ b/freebsd/Platform.c @@ -29,10 +29,6 @@ in the source distribution for its full text. #include <math.h> -#ifndef CLAMP -#define CLAMP(x,low,high) (((x)>(high))?(high):(((x)<(low))?(low):(x))) -#endif - ProcessField Platform_defaultFields[] = { PID, USER, PRIORITY, NICE, M_SIZE, M_RESIDENT, STATE, PERCENT_CPU, PERCENT_MEM, TIME, COMM, 0 }; int Platform_numberOfFields = LAST_PROCESSFIELD; diff --git a/freebsd/Platform.h b/freebsd/Platform.h index fd704539..1a180551 100644 --- a/freebsd/Platform.h +++ b/freebsd/Platform.h @@ -13,10 +13,6 @@ in the source distribution for its full text. extern ProcessFieldData Process_fields[]; -#ifndef CLAMP -#define CLAMP(x,low,high) (((x)>(high))?(high):(((x)<(low))?(low):(x))) -#endif - extern ProcessField Platform_defaultFields[]; extern int Platform_numberOfFields; |