Welp, I just finished commiting and doing a pull request to the main branch with these bug fixes.Gampy wrote:That is exactly what is supposed to happen ... It is followed by ATZ, atz needs to be tested to ensure it returns OK, then the rest of the commands while also checking each for their proper response.
Sadly, I did the pull request incorrectly and it merged all the commits into the main branch instead of just the latest one... I'll have to try again next time. Oh well, it's my test project to try to learn how this crap works lol
It does now send a \r and then verified response from device.
Aaron, can you test the latest version with your OBDLink and let me know what you get? It should also now recognize that you have an OBDLink (i think it'll show up as an STN part, maybe...)
My desktop doesn't have a bluetooth module, so I can't easily test this. All the sleeps have been removed and instead verification of response is done before sending more commands (the proper way).
I also fixed the shutdown bug. As expected, it was indeed a threaded task issue where the forked thread was hanging the main process. I am no UI developer, and don't have much of a concept of how UI threads are supposed to work.
I tagged the latest version as 0.2. Almost useful now...