logger elm327 sketch bug

[hayden-t]

https://github.com/stanleyhuangyc/Freematics/blob/master/firmware_v2/elm327emu/elm327emu.ino

Bug is that you are reading the char twice on incoming BT data, thus loosing data, and breaking the sketch. Below is how it could be instead.

@@ -46,7 +46,7 @@ void loop()
        cmdlen = 0;
      }
    } else if (cmdlen < sizeof(cmd)) {
-      cmd[cmdlen++] = SerialBT.read();
+      cmd[cmdlen++] = c;
    }
  }
}

[stanleyhuangyc]

That's really a shameful bug. :-) Thanks for figuring out.

[hayden-t]

no probs, happy to help.