commit b2eb29a03a7c5691790f89f038ac7b33b2d671ee
author Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com> Sat Dec 08 18:26:58 2018 +0530
committer James Feist <james.feist@linux.intel.com> Wed Dec 12 17:02:29 2018 +0000
tree b70650ffb5af5e163320e60445e21f1f06fda8d4
parent 87d713abb7373e365b1382ddde38cd71c17708ee

Consider nan too as an override value

Sensors can be overridden for testing purpose. Start treating
nan too as an override value. With this commit, the only way
to reset the override is to restart the service / BMC F/W.

Unit-test:
1. Verified that overriden value is taken effect by setting the value
property
2. Tried setting nan to the value property and verified it's still
in overriden state.

Change-Id: I6a931377bdaf918e37057275e1dde93f0fd5aa22
Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>

src/CPUSensor.cpp

diff --git a/src/CPUSensor.cpp b/src/CPUSensor.cpp
index aebaa4c..d54263a 100644
--- a/src/CPUSensor.cpp
+++ b/src/CPUSensor.cpp
@@ -99,7 +99,7 @@
             float nvalue = std::stof(response);
             responseStream.clear();
             nvalue /= CPUSensor::sensorScaleFactor;
-            if (!isnan(overriddenValue))
+            if (overridenState)
             {
                 nvalue = overriddenValue;
             }