summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenny Baumann <BenBE@geshi.org>2020-11-02 21:36:20 +0100
committerBenny Baumann <BenBE@geshi.org>2020-11-02 22:15:01 +0100
commit0e922d4085f0285cf353323fd5a96edae604d5da (patch)
treee90b8301dc32651f0bb5665f634a9bd0e8eb43c6
parentcb8bb12974af25fc845acc008c529eb86835fe6f (diff)
Integrate NAN check into assignment
The check for NAN is kept to avoid relying on implementation details of the CLAMP macro/function
-rw-r--r--dragonflybsd/Platform.c5
-rw-r--r--linux/LinuxProcessList.c5
-rw-r--r--solaris/Platform.c5
3 files changed, 3 insertions, 12 deletions
diff --git a/dragonflybsd/Platform.c b/dragonflybsd/Platform.c
index 5f9f6373..604994c4 100644
--- a/dragonflybsd/Platform.c
+++ b/dragonflybsd/Platform.c
@@ -176,10 +176,7 @@ double Platform_setCPUValues(Meter* this, int cpu) {
percent = v[0] + v[1] + v[2];
}
- percent = CLAMP(percent, 0.0, 100.0);
- if (isnan(percent)) {
- percent = 0.0;
- }
+ percent = isnan(percent) ? 0.0 : CLAMP(percent, 0.0, 100.0);
v[CPU_METER_FREQUENCY] = NAN;
diff --git a/linux/LinuxProcessList.c b/linux/LinuxProcessList.c
index e91666d0..d3416de5 100644
--- a/linux/LinuxProcessList.c
+++ b/linux/LinuxProcessList.c
@@ -1055,10 +1055,7 @@ static bool LinuxProcessList_recurseProcTree(LinuxProcessList* this, const char*
}
float percent_cpu = (lp->utime + lp->stime - lasttimes) / period * 100.0;
- proc->percent_cpu = CLAMP(percent_cpu, 0.0, cpus * 100.0);
- if (isnan(proc->percent_cpu))
- proc->percent_cpu = 0.0;
-
+ proc->percent_cpu = isnan(percent_cpu) ? 0.0 : CLAMP(percent_cpu, 0.0, cpus * 100.0);
proc->percent_mem = (proc->m_resident * CRT_pageSizeKB) / (double)(pl->totalMem) * 100.0;
if (!preExisting) {
diff --git a/solaris/Platform.c b/solaris/Platform.c
index 6159c357..e7a70c2c 100644
--- a/solaris/Platform.c
+++ b/solaris/Platform.c
@@ -200,10 +200,7 @@ double Platform_setCPUValues(Meter* this, int cpu) {
percent = v[0] + v[1] + v[2];
}
- percent = CLAMP(percent, 0.0, 100.0);
- if (isnan(percent)) {
- percent = 0.0;
- }
+ percent = isnan(percent) ? 0.0 : CLAMP(percent, 0.0, 100.0);
v[CPU_METER_FREQUENCY] = NAN;