CarletonURocketry / hysim

A simulation of the hybrid control system which can be run on POSIX platforms.
2 stars 0 forks source link

Added control_client request acknowledgement and changed it's packet sending method to sendmsg #25

Open bskdany opened 5 days ago

bskdany commented 5 days ago

Changes

The code was properly tested, it integrates the hybric-comm specs and works as expected

What is missing

I'm not sure if I have a different .clang-formatter somewhere or someone else is not using the one provided by the repo but I keep having formatting diffs in my commits.

bskdany commented 3 days ago

Created function to flush RECV_BUFFER_SIZE bytes from recv on the pad_server in the following 4 cases:

linguini1 commented 3 days ago
  • received acknowledgement from sender

If we receive acknowledgement from sender, that is bad, but I would prefer that we just read the correct number of bytes from the receive buffer into nowhere (don't use the received bytes). This way we can avoid flushing the entire buffer when we know the payload size of the garbage data we want to clear.