Updates for background hr (InfiniTimeOrg/InfiniTime#2322)

Author: mark9064

main.cpp

@@ -779,15 +779,15 @@ class Framework {
         }
       } else if (key == 'h') {
         if (heartRateController.State() == Pinetime::Controllers::HeartRateController::States::Stopped) {
-          heartRateController.Start();
+          heartRateController.Enable();
         } else if (heartRateController.State() == Pinetime::Controllers::HeartRateController::States::NotEnoughData) {
           heartRateController.Update(Pinetime::Controllers::HeartRateController::States::Running, 10);
         } else {
           uint8_t heartrate = heartRateController.HeartRate();
           heartRateController.Update(Pinetime::Controllers::HeartRateController::States::Running, heartrate + 10);
         }
       } else if (key == 'H') {
-        heartRateController.Stop();
+        heartRateController.Disable();
       } else if (key == 'i') {
         saveScreenshot();
       } else if (key == 'I') {

sim/components/heartrate/HeartRateController.cpp

@@ -12,17 +12,17 @@ void HeartRateController::Update(HeartRateController::States newState, uint8_t h
   }
 }
 
-void HeartRateController::Start() {
+void HeartRateController::Enable() {
   if (task != nullptr) {
     state = States::NotEnoughData;
-    task->PushMessage(Pinetime::Applications::HeartRateTask::Messages::StartMeasurement);
+    task->PushMessage(Pinetime::Applications::HeartRateTask::Messages::Enable);
   }
 }
 
-void HeartRateController::Stop() {
+void HeartRateController::Disable() {
   if (task != nullptr) {
     state = States::Stopped;
-    task->PushMessage(Pinetime::Applications::HeartRateTask::Messages::StopMeasurement);
+    task->PushMessage(Pinetime::Applications::HeartRateTask::Messages::Disable);
   }
 }
 

sim/components/heartrate/HeartRateController.h

@@ -16,8 +16,8 @@ namespace Pinetime {
       enum class States { Stopped, NotEnoughData, NoTouch, Running };
 
       HeartRateController() = default;
-      void Start();
-      void Stop();
+      void Enable();
+      void Disable();
       void Update(States newState, uint8_t heartRate);
 
       void SetHeartRateTask(Applications::HeartRateTask* task);
@@ -37,4 +37,4 @@ namespace Pinetime {
       //Pinetime::Controllers::HeartRateService* service = nullptr;
     };
   }
-}
+}

sim/heartratetask/HeartRateTask.h

@@ -14,22 +14,22 @@ namespace Pinetime {
   namespace Applications {
     class HeartRateTask {
     public:
-      enum class Messages : uint8_t { GoToSleep, WakeUp, StartMeasurement, StopMeasurement };
-      enum class States { Idle, Running };
+      enum class Messages : uint8_t { GoToSleep, WakeUp, Enable, Disable };
 
       explicit HeartRateTask(Drivers::Hrs3300& heartRateSensor, Controllers::HeartRateController& controller);
       void Start();
       void Work();
       void PushMessage(Messages msg);
 
     private:
+      enum class States : uint8_t { Disabled, Waiting, BackgroundMeasuring, ForegroundMeasuring };
       //static void Process(void* instance);
       //void StartMeasurement();
       //void StopMeasurement();
 
 //      TaskHandle_t taskHandle;
       QueueHandle_t messageQueue;
-      States state = States::Running;
+      States state = States::Disabled;
       Drivers::Hrs3300& heartRateSensor;
       Controllers::HeartRateController& controller;
 //      Controllers::Ppg ppg;