Merge pull request #359 from pennam/esp8266-drift-fix

pennam · web-flow · commit 530fd9f63a0b · 2023-05-15T08:43:06.000+02:00
TimeService: Fix esp8266 time drift
diff --git a/src/utility/time/RTCMillis.cpp b/src/utility/time/RTCMillis.cpp
@@ -54,9 +54,8 @@ unsigned long RTCMillis::get()
 {
   unsigned long current_tick = millis();
   unsigned long const elapsed_s = (current_tick - _last_rtc_update_tick) / 1000;
-  if(elapsed_s) {
-    set(_last_rtc_update_value + elapsed_s);
-  }
+  _last_rtc_update_value += elapsed_s;
+  _last_rtc_update_tick += elapsed_s * 1000;
   return _last_rtc_update_value;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -54,9 +54,8 @@ unsigned long RTCMillis::get()`
`54`	`54`	`{`
`55`	`55`	`unsigned long current_tick = millis();`
`56`	`56`	`unsigned long const elapsed_s = (current_tick - _last_rtc_update_tick) / 1000;`
`57`		`- if(elapsed_s) {`
`58`		`- set(_last_rtc_update_value + elapsed_s);`
`59`		`- }`
	`57`	`+ _last_rtc_update_value += elapsed_s;`
	`58`	`+ _last_rtc_update_tick += elapsed_s * 1000;`
`60`	`59`	`return _last_rtc_update_value;`
`61`	`60`	`}`
`62`	`61`