From dcc33bfaa6da25b24a82f75eafb1c88b7edf1499 Mon Sep 17 00:00:00 2001
From: Eric Duminil <eric.duminil@gmail.com>
Date: Sat, 24 Apr 2021 19:39:13 +0200
Subject: [PATCH] co2_sensor: Replace (if + switch) with switch.

---
 ampel-firmware/co2_sensor.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/ampel-firmware/co2_sensor.cpp b/ampel-firmware/co2_sensor.cpp
index 93eb1da..74afdb3 100644
--- a/ampel-firmware/co2_sensor.cpp
+++ b/ampel-firmware/co2_sensor.cpp
@@ -204,8 +204,8 @@ namespace sensor {
     case BOOTUP:
       led_effects::showWaitingLED(color::blue);
       break;
-      // No special signaling, we want to show the CO2 value
     case READY:
+      displayCO2OnLedRing();
       break;
     case NEEDS_CALIBRATION:
       led_effects::showWaitingLED(color::magenta);
@@ -261,12 +261,7 @@ namespace sensor {
       logToSerial();
     }
 
-    // We need to show LED effects for "old" data, too, as long as we get new data.
-    if (current_state == READY) {
-      displayCO2OnLedRing();
-    } else {
-      showState();
-    }
+    showState();
 
     return freshData;
   }
-- 
GitLab