https://github.com/ros2/rmw/pull/277 changed the rmw_service_server_is_available() API documentation to indicate that it returns RMW_RET_INVALID_ARGUMENT if node, client, or is_available are NULL. However, this wasn't the case in practice in the implementations of rmw_service_server_is_available() or in the relevant test.
Change the implementation here to return RMW_RET_INVALID_ARGUMENT.
christophebedard
commented
5 months ago
https://github.com/ros2/rmw/pull/277 changed the
rmw_service_server_is_available()API documentation to indicate that it returnsRMW_RET_INVALID_ARGUMENTifnode,client, oris_availableareNULL. However, this wasn't the case in practice in the implementations ofrmw_service_server_is_available()or in the relevant test.Change the implementation here to return
RMW_RET_INVALID_ARGUMENT.Requires https://github.com/ros2/rmw_implementation/pull/231