This pull request fixes #12. The root cause was [`BLE_HandleCommonEvents()`](https://git.ktag.clubk.club/Software/SystemK/src/tag/SystemK_v01.01/BLE/BLE_Utils.c#L57) was causing BLE events to be marked as "not new" by [`BLE_IsPacketNew()`](https://git.ktag.clubk.club/Software/SystemK/src/tag/SystemK_v01.01/BLE/BLE_Packet_Tracker.c#L53), even though these packets were left unhandled by the common event handler. The solution is to separate the querying of whether a packet is new from the marking it as already seen, since we don't want to mark a packet as seen until it has been properly _handled_. I added the function `BLE_RememberPacket()` for the latter purpose. I also cleaned up some of the logging code, now that logging is configurable (on ESP32 platform, anyway). Co-authored-by: Joe Kearney <joe@clubk.club> Reviewed-on: #13 |
||
|---|---|---|
| .. | ||
| Playing | ||
| Starting_Game | ||
| State_Configuring.c | ||
| State_Configuring.h | ||
| State_Initializing.c | ||
| State_Initializing.h | ||
| State_Machine.c | ||
| State_Machine.h | ||
| State_Ready.c | ||
| State_Ready.h | ||
| State_Reprogramming.c | ||
| State_Reprogramming.h | ||
| State_Wrapping_Up.c | ||
| State_Wrapping_Up.h | ||