command sender c code dodgyness

[alan-stokes]

Lines 142, 150, 164, 169 dont check for success on the sending of a mc packet.

this is dodgy. given the real reason for repeats was for the external device, not for us to try 5 times without checking at least 1 got throguh.

[rowleya]

Note that passing a packet through the send routine is no guarantee of success in sending; I think that just means that it will make it in to the router buffer for sending (still a bit better than not sending at all). The router can still drop packets if the other end isn't listening. So sending repeats is still useful even if the send is "successful".

[alan-stokes]

many many places it can go wrong. but we might as well remove 1 of them if we can