rusefi
/
custom-board-bundle-sample-fork-of-rusefi
mirror of https://github.com/rusefi/custom-board-bundle-sample-fork-of-rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

efiWatchdog: avoid converting time to float (#4562) 

https://stackoverflow.com/questions/68690961/why-does-converting-int-to-float-lose-precision-in-c-programming-language
This commit is contained in:
Andrey G 2022-09-11 12:46:55 +03:00 committed by GitHub
parent 2e85a7a09d
commit c73a8c2f46
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
firmware/controllers/algo/engine.cpp
View File

@ -454,13 +454,12 @@ void Engine::efiWatchdog() {
if (engine->configBurnTimer.hasElapsedSec(5) && engineConfiguration->tempBooleanForVerySpecialLogic) {
static efitimems_t mostRecentMs = 0;
float msNow = currentTimeMillis();
if (msNow != 0) {
float gapInMs = msNow - mostRecentMs;
efitimems_t msNow = currentTimeMillis();
if (mostRecentMs != 0) {
efitimems_t gapInMs = msNow - mostRecentMs;
if (gapInMs > 500) {
// float has 24 bits in the mantissa, which should allow up to 8 significant figures
// we loose precision here after about 1,000,000 seconds which is 11 days
firmwareError(WATCH_DOG_SECONDS, "gap in time: now=%fms gap=%fms", msNow, gapInMs);
firmwareError(WATCH_DOG_SECONDS, "gap in time: now=%d mS, was %d mS, gap=%dmS",
msNow, mostRecentMs, gapInMs);
}
}
mostRecentMs = msNow;