OpenCyphal / libcanard

A compact implementation of the Cyphal/CAN protocol in C for high-integrity real-time embedded systems
http://opencyphal.org
MIT License
332 stars 192 forks source link

Bug: The `.timestamp_usec` should not be a zero. #159

Closed Linjieqiang closed 3 years ago

Linjieqiang commented 3 years ago

image

The .timestamp_usec should not be a zero, otherwise it will not be sent normally, because the condition if (txf->timestamp_usec > getCurrentMicroseconds()) is not met. Maybe it should be changed to: micros() + CANARD_DEFAULT_TRANSFER_ID_TIMEOUT_USEC".

pavel-kirienko commented 3 years ago

@Linjieqiang please review https://github.com/UAVCAN/libcanard/pull/160