Use action client get result method

[orduno]

Multiple places use the ClientGoalHandle.async_result() to get the action result. However, it seems safer to use Client.async_get_result(), plus that way we don't have to provide an empty lambda function to make it result aware.

That seems to be the intended use.

[orduno]

This could be a possible source of errors, so I'm going to promote it high.