Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.Sign up
Reply parser was not honouring replies with multiple items #4965
ECoS state machine is updated on replies from the command station.
The response parser was only considering single item responses.
As a result of the change introduced in #4945 we now generate messages containing multiple items (speed and dir).
This PR fixes the reply parser to correctly update the state machine from such messages. Without this change, only speed was being updated by these multi-item responses.
In addition, there is some clean-up of unnecessary Float boxing and additions to some missing JavaDoc - both highlighted by NetBeans.
Travis headless failed due to: