2 files changed, 28 insertions, 27 deletions

diff --git a/src/components/timer/TimerController.cpp b/src/components/timer/TimerController.cpp
index 79e44d6..d6f8ffd 100644
--- a/src/components/timer/TimerController.cpp
+++ b/src/components/timer/TimerController.cpp
@@ -9,18 +9,16 @@
 
 using namespace Pinetime::Controllers;
 
-
 APP_TIMER_DEF(timerAppTimer);
 
 namespace {
   void TimerEnd(void* p_context) {
-    auto* controller = static_cast<Pinetime::Controllers::TimerController*> (p_context);
-    if(controller != nullptr)
+    auto* controller = static_cast<Pinetime::Controllers::TimerController*>(p_context);
+    if (controller != nullptr)
       controller->OnTimerEnd();
   }
 }
 
-
 void TimerController::Init() {
   app_timer_create(&timerAppTimer, APP_TIMER_MODE_SINGLE_SHOT, TimerEnd);
 }
@@ -31,37 +29,37 @@ void TimerController::StartTimer(uint32_t duration) {
   app_timer_start(timerAppTimer, APP_TIMER_TICKS(duration), this);
   endTicks = currentTicks + APP_TIMER_TICKS(duration);
   timerRunning = true;
+  overtime = false;
 }
 
-uint32_t TimerController::GetTimeRemaining() {
+int32_t TimerController::GetSecondsRemaining() {
   if (!timerRunning) {
     return 0;
   }
   auto currentTicks = xTaskGetTickCount();
-  
-  TickType_t deltaTicks = 0;
-  if (currentTicks > endTicks) {
-    deltaTicks = 0xffffffff - currentTicks;
-    deltaTicks += (endTicks + 1);
-  } else {
-    deltaTicks = endTicks - currentTicks;
-  }
-  
-  return (static_cast<TickType_t>(deltaTicks) / static_cast<TickType_t>(configTICK_RATE_HZ)) * 1000;
+
+  int32_t deltaTicks = static_cast<int32_t>(endTicks) - static_cast<int32_t>(currentTicks);
+
+  return (deltaTicks / static_cast<int32_t>(configTICK_RATE_HZ));
 }
 
 void TimerController::StopTimer() {
   app_timer_stop(timerAppTimer);
   timerRunning = false;
+  overtime = false;
 }
 
-bool TimerController::IsRunning() {
-  return timerRunning;
+void TimerController::StopAlerting() {
+  if (systemTask != nullptr) {
+    systemTask->PushMessage(System::Messages::StopRinging);
+  }
 }
+
 void TimerController::OnTimerEnd() {
-  timerRunning = false;
-  if(systemTask != nullptr)
+  overtime = true;
+  if (systemTask != nullptr) {
     systemTask->PushMessage(System::Messages::OnTimerDone);
+  }
 }
 
 void TimerController::Register(Pinetime::System::SystemTask* systemTask) {
diff --git a/src/components/timer/TimerController.h b/src/components/timer/TimerController.h
index fa7bc90..dbc9863 100644
--- a/src/components/timer/TimerController.h
+++ b/src/components/timer/TimerController.h
@@ -9,20 +9,22 @@ namespace Pinetime {
     class SystemTask;
   }
   namespace Controllers {
-    
+
     class TimerController {
     public:
       TimerController() = default;
-      
+
       void Init();
-      
       void StartTimer(uint32_t duration);
-      
       void StopTimer();
-      
-      uint32_t GetTimeRemaining();
-      
-      bool IsRunning();
+      void StopAlerting();
+      int32_t GetSecondsRemaining();
+      bool IsOvertime() {
+        return overtime;
+      }
+      bool IsRunning() {
+        return timerRunning;
+      }
 
       void OnTimerEnd();
 
@@ -32,6 +34,7 @@ namespace Pinetime {
       System::SystemTask* systemTask = nullptr;
       TickType_t endTicks;
       bool timerRunning = false;
+      bool overtime = false;
     };
   }
 }
\ No newline at end of file