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/ChassisIntrusionSensor.cpp b/src/ChassisIntrusionSensor.cpp
index 4b2327e..15ed888 100644
--- a/src/ChassisIntrusionSensor.cpp
+++ b/src/ChassisIntrusionSensor.cpp
@@ -47,6 +47,13 @@
void ChassisIntrusionSensor::updateValue(const std::string newValue)
{
+ // Take no action if value already equal
+ // Same semantics as Sensor::updateValue(const double&)
+ if (newValue == mValue)
+ {
+ return;
+ }
+
// indicate that it is internal set call
mInternalSet = true;
mIface->set_property("Status", newValue);