Sorry for the excessive posting but I parsed the 0.9.13 source with cppcheck and it found an out of bounds error in the est_isr() and a missing address of operator in getDwell(). Fixes are on github as 0.9.14. Also cleaned up some preprocessor warnings about narrowing casts. This build is untested but I will try to test base functionality (starts, runs, etc.) tonight. EDIT: basic functionality tests were good.
By the way, one added feature in this build that I overlooked in my previous post is the theoretical ability to handle ATDC (aka negative advance) spark timing. I'm not sure how to test this in-vehicle - possibly wait for the next ice storm and go out for a short drive to trip the ABS.
Bookmarks