search

ros2 / rmw_fastrtps

Implementation of the ROS Middleware (rmw) Interface using eProsima's Fast RTPS.
Apache License 2.0
147 stars 116 forks source link

Make rmw_destroy_wait_set return RMW_RET_INVALID_ARGUMENT #766

Closed christophebedard closed 2 weeks ago

christophebedard commented 4 weeks ago

https://github.com/ros2/rmw/pull/275 changed the rmw_destroy_wait_set() API documentation to indicate that it returns RMW_RET_INVALID_ARGUMENT if wait_set is NULL. However, this wasn't the case in practice in the implementations of rmw_destroy_wait_set() or in the relevant test.

Change the implementation here to return RMW_RET_INVALID_ARGUMENT.

Requires https://github.com/ros2/rmw_implementation/pull/234

christophebedard commented 4 weeks ago

CI is over in https://github.com/ros2/rmw_implementation/pull/234#issuecomment-2150237344