diff options
author | Benny Baumann <BenBE@geshi.org> | 2020-11-02 21:36:20 +0100 |
---|---|---|
committer | Benny Baumann <BenBE@geshi.org> | 2020-11-02 22:15:01 +0100 |
commit | 0e922d4085f0285cf353323fd5a96edae604d5da (patch) | |
tree | e90b8301dc32651f0bb5665f634a9bd0e8eb43c6 | |
parent | cb8bb12974af25fc845acc008c529eb86835fe6f (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.c | 5 | ||||
-rw-r--r-- | linux/LinuxProcessList.c | 5 | ||||
-rw-r--r-- | solaris/Platform.c | 5 |
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; |