Found instant comparison bug

FluenTech / embedded-time

Time(ing) library (Instant/Duration/Clock/Timer/Period/Frequency) for bare-metal embedded systems

Apache License 2.0

87 stars 17 forks source link

Found instant comparison bug #120

Closed korken89 closed 2 years ago

korken89 commented 2 years ago

Hi @PTaylor-us

I found a bug in the comparisons, see attached PR that causes CI to fail. We found this because the scheduling in RTIC was always of by 1.

We should fix this before 0.13 release.

korken89 commented 2 years ago

I fixed the bug

PTaylor-us commented 2 years ago

Thank you! After a cursory glance at the fix, I can't quite tell why it was failing. I will get this merged and released in a point release in the next day or two.