time: fix millis wrap around in delay()

pennam · pennam · commit 93f2368f3b4d · 2025-12-11T09:43:58.000+01:00
diff --git a/cores/arduino/time.cpp b/cores/arduino/time.cpp
@@ -6,9 +6,8 @@
 
 __attribute__((weak)) void delay(uint32_t ms)
 {
-  auto const start = millis();
-  auto const stop = start + ms;
-  while(millis() < stop) yield();
+  uint32_t const start = millis();
+  while(millis() - start < ms) yield();
 }
 
 void delayMicroseconds(unsigned int us) {

Original file line number	Diff line number	Diff line change
`@@ -6,9 +6,8 @@`
`6`	`6`
`7`	`7`	`__attribute__((weak)) void delay(uint32_t ms)`
`8`	`8`	`{`
`9`		`- auto const start = millis();`
`10`		`- auto const stop = start + ms;`
`11`		`- while(millis() < stop) yield();`
	`9`	`+ uint32_t const start = millis();`
	`10`	`+ while(millis() - start < ms) yield();`
`12`	`11`	`}`
`13`	`12`
`14`	`13`	`void delayMicroseconds(unsigned int us) {`