-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Migrate TransformBroadcaster to use shared_ptr #5569
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
Migrate TransformBroadcaster to use shared_ptr #5569
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For Kilted/Jazzy support these should be #ifdef ed like the other TF message filter work for API changes so we can support all on the branch
78b9d6f to
693a327
Compare
|
Why do these seem to work?
Maybe just needs |
693a327 to
c879dbd
Compare
Apologies for the oversight. We can use this approach instead of the stack allocation earlier. |
Signed-off-by: Leander Stephen D'Souza <[email protected]>
c879dbd to
d281630
Compare
…#5569) Signed-off-by: Leander Stephen D'Souza <[email protected]>
…#5569) Signed-off-by: Leander Stephen D'Souza <[email protected]>
…#5569) Signed-off-by: Leander Stephen D'Souza <[email protected]>
Basic Info
system_buildjobDescription of contribution in a few bullet points
TransformBroadcasterenforced in ros2/geometry2.Description of documentation updates required from your changes
Description of how this change was tested
nav2_routenow passes locally -colcon build --packages-select nav2_route.Future work that may be required in bullet points
For Maintainers:
backport-*.