broadcast button toggle counter #5514

only:fancy boolen with counter

firmware/controllers/algo/engine.cpp

@@ -281,6 +281,9 @@ Engine::Engine()
 #if EFI_ANTILAG_SYSTEM
 	, ALSsoftSparkLimiter(false)
 #endif /* EFI_ANTILAG_SYSTEM */
+	, clutchUpSwitchedState(&engineState.clutchUpState)
+	, brakePedalSwitchedState(&engineState.brakePedalState)
+	, acButtonSwitchedState(&module<AcController>().unmock().acButtonState)
 
 #endif // EFI_LAUNCH_CONTROL
 {

firmware/controllers/algo/engine.h

@@ -50,6 +50,7 @@
 #include "throttle_model.h"
 #include "gc_generic.h"
 #include "lambda_monitor.h"
+#include "efi_output.h"
 
 #ifndef EFI_UNIT_TEST
 #error EFI_UNIT_TEST must be defined!
@@ -337,6 +338,10 @@ public:
 	AirmassModelBase* mockAirmassModel = nullptr;
 #endif
 
+	SwitchedState clutchUpSwitchedState;
+	SwitchedState brakePedalSwitchedState;
+	SwitchedState acButtonSwitchedState;
+
 private:
 	void reset();
 

firmware/controllers/system/efi_output.cpp

@@ -3,11 +3,20 @@
  *
  */
 
+#include "pch.h"
 #include "efi_output.h"
 
-void SwitchState::update(bool newState) {
+
+bool SwitchedState::update(bool newState) {
     if (newState != *state) {
-        *state = newState;
+        *state = newState ? 1 : 0;
         counter++;
+        return true;
     }
+    return false;
 }
+
+uint16_t SwitchedState::getCounter() {
+	return counter;
+}
+

firmware/controllers/system/efi_output.h

@@ -3,20 +3,30 @@
  *
  */
 
+#pragma once
+
 #include "io_pins.h"
 #include "smart_gpio.h"
 
-#pragma once
 
-class SwitchState {
+// This class acts as a boolean, but has a switch counter inside
-    void init(bool *state, uint16_t *counter) {
+class SwitchedState {
+public:
+    SwitchedState(int8_t *state) {
         this->state = state;
-        this->counter = counter;
     }
 
-    bool *state;
+    // returns true if the state has been changed
-    uint16_t *counter;
+    bool update(bool newState);
-    void update(bool newState);
+    uint16_t getCounter();
+
+    operator bool() const {
+        return (bool)*state;
+    }
+
+private:
+    int8_t *state;
+    uint16_t counter = 0;
 };
 
 // Used if you want a function to be virtual only for unit testing purposes