Cleaned up usage of updateValue() callers
Removed needless float, simply using double
Corrected string parsing function called for each type
Callers no longer need to check for equality first
Equality test now added to int and string override methods
This ensures all have same semantics
Removed extraordinarily spammy debug in updateValue(double)
Signed-off-by: Josh Lehan <krellan@google.com>
Change-Id: If8422d6516e083041685ddb6ef8cd469299cbad3
diff --git a/src/CPUSensor.cpp b/src/CPUSensor.cpp
index 27c9c38..d3789fd 100644
--- a/src/CPUSensor.cpp
+++ b/src/CPUSensor.cpp
@@ -114,20 +114,19 @@
try
{
std::getline(responseStream, response);
- double nvalue = std::stof(response);
+ double nvalue = std::stod(response);
responseStream.clear();
nvalue /= CPUSensor::sensorScaleFactor;
- if (nvalue != value)
+
+ if (show)
{
- if (show)
- {
- updateValue(nvalue);
- }
- else
- {
- value = nvalue;
- }
+ updateValue(nvalue);
}
+ else
+ {
+ value = nvalue;
+ }
+
double gTcontrol = gCpuSensors[nameTcontrol]
? gCpuSensors[nameTcontrol]->value
: std::numeric_limits<double>::quiet_NaN();