lexus2k / tinyproto / 23976259378

90%

push

github

Fix C++ wrapper reliability issues, add 19 unit tests

Bug fixes:
- Light::write(IPacket): return actual byte count instead of bool (was
  converting result > 0, destroying error info and always returning 0 or 1)
- IFd::run_tx(write_func): break on write returning 0 to prevent infinite
  loop when device is busy (was spinning forever on len -= 0)
- IFd::run_rx/run_tx: increase internal buffer from 4 to 64 bytes for
  better throughput on high-speed serial connections
- IHdlcLinkLayer::getData(): protect m_flushFlag read with m_sendMutex
  to fix race condition with flushTx() from another thread
- ISerialLinkLayer::runTx(): break on write returning 0 (same infinite
  loop fix as IFd::run_tx)
- FdD destructor: store original uintptr_t* pointer to fix undefined
  behavior from delete[] on mismatched type (was allocating uintptr_t[]
  but deleting as uint8_t[])
- Proto::end(): add null check for m_link before calling m_link->end()
- Proto::getLostRxFrames(): protect counter access with mutex

New tests (19 added, 101 total):
- CPP_LIGHT (3): WriteReturnsByteCount, WritePacketReturnsByteCount,
  SendAndReceive
- CPP_HDLC (3): WriteAndGenerateTxData, SendAndReceiveFrame,
  WritePacketInterface
- CPP_FD (10): BeginAndEnd, StaticBufferVariant, DynamicAllocationCleanup,
  WriteReturnsProperValues, RunRxWithDirectData, RunTxGeneratesData,
  RunTxCallbackBreaksOnZeroWrite, ReceiveCallback, ConnectEventCallback,
  CrcModes
- CPP_PACKET (3): PutAndRead, CopyConstructor, Clear

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

3 of 3 new or added lines in 2 files covered. (100.0%)

1228 of 1365 relevant lines covered (89.96%)

8400.48 hits per line