commit 3ebaa3e6de4a28ab5b19e357e5f334dd2cd54b86
author Patrick Venture <venture@google.com> Fri Jun 21 12:15:18 2019 -0700
committer Patrick Venture <venture@google.com> Fri Jun 21 12:15:18 2019 -0700
tree 299fc5135c76cc40583c4ec74dd345076ba0d030
parent 6206723d6cd8a7fa678cedae0066b3dec4b6297e

sensor: remove redundant condition

[sensor.cpp:147]: (style) Redundant condition: statusIface.
'!statusIface || (statusIface && statusIface->functional())' is
equivalent to '!statusIface || statusIface->functional()'

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I4f3c3f6e2e009dfaffc5671dc94b2f553f7029ef

sensor.cpp

diff --git a/sensor.cpp b/sensor.cpp
index e6e8413..a9e4454 100644
--- a/sensor.cpp
+++ b/sensor.cpp
@@ -144,7 +144,7 @@
 
     // If there's no fault file or the sensor has a fault file and
     // its status is functional, read the input value.
-    if (!statusIface || (statusIface && statusIface->functional()))
+    if (!statusIface || statusIface->functional())
     {
         // RAII object for GPIO unlock / lock
         GpioLock gpioLock(getGpio());