Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix deprecated subscriber callback warnings #852

Merged
merged 1 commit into from
Aug 24, 2021
Merged

Fix deprecated subscriber callback warnings #852

merged 1 commit into from
Aug 24, 2021

Conversation

aprotyas
Copy link
Member

ros2/rclcpp#1713 deprecates the void shared_ptr<T> subscription callback signatures, so this PR migrates away from said signatures.

Signed-off-by: Abrar Rahman Protyasha abrar@openrobotics.org

Signed-off-by: Abrar Rahman Protyasha <abrar@openrobotics.org>
@aprotyas aprotyas requested a review from a team as a code owner August 20, 2021 07:08
@aprotyas aprotyas requested review from Karsten1987 and prajakta-gokhale and removed request for a team August 20, 2021 07:08
@aprotyas
Copy link
Member Author

CI (build/test: --packages-above-and-dependencies rosbag2_transport)

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@emersonknapp emersonknapp merged commit 4b71b14 into ros2:master Aug 24, 2021
WJaworskiRobotec pushed a commit to RobotecAI/rosbag2 that referenced this pull request Sep 12, 2021
Signed-off-by: Abrar Rahman Protyasha <abrar@openrobotics.org>
Signed-off-by: Wojciech Jaworski <wojciech.jaworski@robotec.ai>
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.

None yet

3 participants