From 4daaf656949a473d19cb5f4106ffa9a9cf038813 Mon Sep 17 00:00:00 2001
From: Michele Bini <michele.bini@gmail.com>
Date: Tue, 7 Jun 2022 00:32:23 +0200
Subject: use tick-based duration type for computation


diff --git a/src/components/alarm/AlarmController.cpp b/src/components/alarm/AlarmController.cpp
index 596bcaf..94914bd 100644
--- a/src/components/alarm/AlarmController.cpp
+++ b/src/components/alarm/AlarmController.cpp
@@ -19,6 +19,12 @@
 #include "systemtask/SystemTask.h"
 #include "task.h"
 #include <chrono>
+#include <ratio>
+#include <cstdint>
+
+namespace {
+  typedef std::chrono::duration< int64_t, std::ratio<1, configTICK_RATE_HZ> > ticks_t;
+}
 
 using namespace Pinetime::Controllers;
 using namespace std::chrono_literals;
@@ -44,22 +50,21 @@ void AlarmController::SetAlarmTime(uint8_t alarmHr, uint8_t alarmMin) {
 }
 
 void AlarmController::OnAdjustTime() {
-
   if (state != AlarmState::Set) {
     return;
   }
 
-  xTimerStop(alarmAppTimer, 0);
+  xTimerStop(alarmTimer, 0);
   auto now = dateTimeController.CurrentDateTime();
 
-  if (now > alarmTime) {
-    xTimerChangePeriod(alarmAppTimer, 1, 0);
+  if (now >= alarmTime) {
+    xTimerChangePeriod(alarmTimer, 1, 0);
   } else {
-    auto mSecToAlarm = std::chrono::duration_cast<std::chrono::milliseconds>(alarmTime - now).count();
-    xTimerChangePeriod(alarmAppTimer, APP_TIMER_TICKS(mSecToAlarm), 0);
+    auto ticksToAlarm = std::chrono::duration_cast<ticks_t>(alarmTime - now).count();
+    xTimerChangePeriod(alarmTimer, ++ticksToAlarm, 0);
   }
 
-  xTimerStart(alarmAppTimer, 0);
+  xTimerStart(alarmTimer, 0);
 }
 
 void AlarmController::ScheduleAlarm() {
@@ -94,8 +99,8 @@ void AlarmController::ScheduleAlarm() {
 
   // now can convert back to a time_point
   alarmTime = std::chrono::system_clock::from_time_t(std::mktime(tmAlarmTime));
-  auto secondsToAlarm = std::chrono::duration_cast<std::chrono::seconds>(alarmTime - now).count();
-  xTimerChangePeriod(alarmTimer, secondsToAlarm * configTICK_RATE_HZ, 0);
+  auto ticksToAlarm = std::chrono::duration_cast<ticks_t>(alarmTime - now).count();
+  xTimerChangePeriod(alarmTimer, ticksToAlarm, 0);
   xTimerStart(alarmTimer, 0);
 
   state = AlarmState::Set;
-- 
cgit v0.10.2