From e3af69a39edc2a1a6b9025414177fbda223af85b Mon Sep 17 00:00:00 2001
From: Eric Duminil <eric.duminil@gmail.com>
Date: Sat, 9 Jan 2021 00:17:39 +0100
Subject: [PATCH] Removing old todos

---
 ampel-firmware/csv_writer.cpp | 1 -
 ampel-firmware/mqtt.cpp       | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/ampel-firmware/csv_writer.cpp b/ampel-firmware/csv_writer.cpp
index 81b7278..26c0b00 100644
--- a/ampel-firmware/csv_writer.cpp
+++ b/ampel-firmware/csv_writer.cpp
@@ -160,7 +160,6 @@ namespace csv_writer {
 
   void logIfTimeHasCome(const String &timeStamp, const int16_t &co2, const float &temperature, const float &humidity) {
     unsigned long now = seconds();
-    //TODO: Write average since last CSV write?
     if (now - last_written_at > config::csv_interval) {
       last_written_at = now;
       log(timeStamp, co2, temperature, humidity);
diff --git a/ampel-firmware/mqtt.cpp b/ampel-firmware/mqtt.cpp
index 29f609f..4f189bb 100644
--- a/ampel-firmware/mqtt.cpp
+++ b/ampel-firmware/mqtt.cpp
@@ -177,7 +177,7 @@ namespace mqtt {
   }
 
   void reconnect() {
-    if (last_failed_at > 0 && seconds() - last_failed_at < config::wait_after_fail) {
+    if (last_failed_at > 0 && (seconds() - last_failed_at < config::wait_after_fail)) {
       // It failed less than wait_after_fail ago. Not even trying.
       return;
     }
@@ -216,7 +216,6 @@ namespace mqtt {
   void publishIfTimeHasCome(const String &timeStamp, const int16_t &co2, const float &temp, const float &hum) {
     // Send message via MQTT according to sending interval
     unsigned long now = seconds();
-    //TODO: Send average since last MQTT message?
     if (now - last_sent_at > config::sending_interval) {
       last_sent_at = now;
       publish(timeStamp, co2, temp, hum);
-- 
GitLab