From 8d34196a15b8b8c2819ab56fd2a0f2f6c033f7aa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc=20Poulhi=C3=A8s?= <github@kataplop.net>
Date: Thu, 6 Jul 2017 07:05:05 +0200
Subject: [PATCH] use same time measure for whole function

should have minor impact, but use same time value for all computation.
---
 arduino/startracker.ino | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arduino/startracker.ino b/arduino/startracker.ino
index 4b4954d..20748d7 100644
--- a/arduino/startracker.ino
+++ b/arduino/startracker.ino
@@ -210,18 +210,20 @@ static void handle_active_timer(void){
   Serial.print("Timer: ");
   Serial.println(active_timer.remain);
 #endif
+  unsigned long current_time = millis();
 
   if (active_timer.deadline){
-    active_timer.remain = active_timer.deadline - millis();
+    active_timer.remain = active_timer.deadline - current_time;
   } else {
     return;
   }
 
   if (active_timer.remain <= active_threshold) {
     active_timer.remain = active_timer.period;
-    active_timer.deadline = millis() + active_timer.period;
+    active_timer.deadline = current_time + active_timer.period;
     active_timer.expired++;
     loop_count++;
+
 #if DEBUG
     Serial.println("Timer expired");
     Serial.println(active_timer.expired);