- Motivation: The shared pointer to the service event type support
library shall be a member variable to make sure that library loaded
during the liveliness of the instance of this class, since we have a
raw pointers to its inner members.

Signed-off-by: Michael Orlov <[email protected]>

- Rewrite raw pointers arithmetic. Assumption that size_t represent size
of the void* may not necessarily be true on all platforms.
- Use shared pointer with custom deleter for deserialized message.
- Assumption that we can take first element from bounded sequence by
dereferencing raw pointer to the bounded sequence may not be necessarily
be true and up to the underlying rmw and transport layer implementation.
Use dedicated request_member.get_function(request_sequence_ptr, 0)
function instead.
- Add sanity checks `for service_event_members_` in PlayerServiceClient
constructor.

Signed-off-by: Michael Orlov <[email protected]>

- Rationale: We can't rely on assumption that we can safely partially
deserialize service event to the ServiceEventInfo structure.

Signed-off-by: Michael Orlov <[email protected]>

…ustify errors

Signed-off-by: Barry Xu <[email protected]>

- Rationale: We are moving to the new version of the uncrustify in
rolling and still haven't done it yet fully for the baseline.
The discrepancy in style for other untouched files like logging.hpp
shall be addressed in a separate PR.

Signed-off-by: Michael Orlov <[email protected]>

- Also rename enum class ServiceRequestFrom to the ServiceRequestsSource

Signed-off-by: Michael Orlov <[email protected]>

- We need this API to be able to write deterministic and non-flaky tests

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Michael Orlov <[email protected]>

- Get rid of timeout inside tests and use newly added
player->wait_for_sent_service_requests_to_finish(service_name) API.

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Michael Orlov <[email protected]>

Also added a new option publish_service_requests to the PlayOptions.
Note: By default rosbag2 player will publish service events only.

Signed-off-by: Barry Xu <[email protected]>
Signed-off-by: Michael Orlov <[email protected]>

Co-authored-by: Barry Xu <[email protected]>
Signed-off-by: Michael Orlov <[email protected]>

- Long story short: Make it deterministic and run fast.

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Michael Orlov <[email protected]>

Signed-off-by: Barry Xu <[email protected]>