Integrate NAN check into assignment

The check for NAN is kept to avoid relying on implementation details of the CLAMP macro/function
This commit is contained in:
Benny Baumann 2020-11-02 21:36:20 +01:00
parent cb8bb12974
commit 0e922d4085
3 changed files with 3 additions and 12 deletions

View File

@ -176,10 +176,7 @@ double Platform_setCPUValues(Meter* this, int cpu) {
percent = v[0] + v[1] + v[2]; percent = v[0] + v[1] + v[2];
} }
percent = CLAMP(percent, 0.0, 100.0); percent = isnan(percent) ? 0.0 : CLAMP(percent, 0.0, 100.0);
if (isnan(percent)) {
percent = 0.0;
}
v[CPU_METER_FREQUENCY] = NAN; v[CPU_METER_FREQUENCY] = NAN;

View File

@ -1055,10 +1055,7 @@ static bool LinuxProcessList_recurseProcTree(LinuxProcessList* this, const char*
} }
float percent_cpu = (lp->utime + lp->stime - lasttimes) / period * 100.0; float percent_cpu = (lp->utime + lp->stime - lasttimes) / period * 100.0;
proc->percent_cpu = CLAMP(percent_cpu, 0.0, cpus * 100.0); proc->percent_cpu = isnan(percent_cpu) ? 0.0 : CLAMP(percent_cpu, 0.0, cpus * 100.0);
if (isnan(proc->percent_cpu))
proc->percent_cpu = 0.0;
proc->percent_mem = (proc->m_resident * CRT_pageSizeKB) / (double)(pl->totalMem) * 100.0; proc->percent_mem = (proc->m_resident * CRT_pageSizeKB) / (double)(pl->totalMem) * 100.0;
if (!preExisting) { if (!preExisting) {

View File

@ -200,10 +200,7 @@ double Platform_setCPUValues(Meter* this, int cpu) {
percent = v[0] + v[1] + v[2]; percent = v[0] + v[1] + v[2];
} }
percent = CLAMP(percent, 0.0, 100.0); percent = isnan(percent) ? 0.0 : CLAMP(percent, 0.0, 100.0);
if (isnan(percent)) {
percent = 0.0;
}
v[CPU_METER_FREQUENCY] = NAN; v[CPU_METER_FREQUENCY] = NAN;