Skip to content

Correct the incorrect comments in generic_client.hpp#2662

Merged
clalancette merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:review/topic-fix-comments-in-generic-client
Nov 5, 2024
Merged

Correct the incorrect comments in generic_client.hpp#2662
clalancette merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:review/topic-fix-comments-in-generic-client

Conversation

@Barry-Xu-2018
Copy link
Copy Markdown
Collaborator

This is a minor fix.

Since the current rmw does not provide an interface to obtain serialized request and response data, both generic_client and generic_service are currently handling deserialized untyped data.

In the code comments, it was mistakenly written as serialized data.

Signed-off-by: Barry Xu <barry.xu@sony.com>
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Nov 5, 2024

Pulls: #2662
Gist: https://gist.githubusercontent.com/ahcorde/b478acc0a16df1d9e7ffe5f2cc2d29c7/raw/0b7c7c9a92583cbd5bb511ecd21d5bfddd1f146e/ros2.repos
BUILD args: --packages-up-to rclcpp --packages-above-and-dependencies rclcpp
TEST args: --packages-select rclcpp --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14782

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants