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

Use node interface pointers in the MessageFilter class #96

Merged
merged 2 commits into from
Mar 13, 2019
Merged

Use node interface pointers in the MessageFilter class #96

merged 2 commits into from
Mar 13, 2019

Conversation

mjeronimo
Copy link

In the MessageFilter class, use node interfaces instead of using the node directly so that the code works with either rclcpp::Node or rclcpp_lifecycle::LifecycleNode. Retain the existing node-based interface for backwards compatibility.

Michael Jeronimo added 2 commits March 6, 2019 09:56
In the MessageFilter class, use node interfaces instead of using the node directly
so that the code works with either rclcpp::Node or rclcpp_lifecycle::LifecycleNode.
Retain the existing node-based interface for backwards compatibility.
@tfoote tfoote added the in review Waiting for review (Kanban column) label Mar 6, 2019
Copy link
Contributor

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, kicking off CI

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

@mjeronimo
Copy link
Author

@tfoote Is this one OK to be integrated?

@tfoote tfoote merged commit 00faaf1 into ros2:ros2 Mar 13, 2019
@tfoote tfoote removed the in review Waiting for review (Kanban column) label Mar 13, 2019
nuclearsandwich added a commit to ros/rosdistro that referenced this pull request Apr 14, 2019
Increasing version of package(s) in repository `geometry2` to `0.11.0-1`:

- upstream repository: https://github.com/ros2/geometry2.git
- release repository: https://github.com/ros2-gbp/geometry2-release.git
- distro file: `dashing/distribution.yaml`
- bloom version: `0.8.0`
- previous version for package: `null`

## tf2

- No changes

## tf2_eigen

```
* Updated to use ament_target_dependencies where possible. (#98 <ros2/geometry2#98>)
* Contributors: ivanpauno
```

## tf2_geometry_msgs

```
* Updated to use ament_target_dependencies where possible. (#98 <ros2/geometry2#98>)
```

## tf2_msgs

- No changes

## tf2_ros

```
* Updated to use node inteface pointers in the MessageFilter class. (#96 <ros2/geometry2#96>)
* Updated message_filter.h. (#91 <ros2/geometry2#91>)
* Contributors: Michael Jeronimo, Sagnik Basu
```

## tf2_sensor_msgs

- No changes
andre-rosa pushed a commit to andre-rosa/rosdistro that referenced this pull request May 3, 2019
Increasing version of package(s) in repository `geometry2` to `0.11.0-1`:

- upstream repository: https://github.com/ros2/geometry2.git
- release repository: https://github.com/ros2-gbp/geometry2-release.git
- distro file: `dashing/distribution.yaml`
- bloom version: `0.8.0`
- previous version for package: `null`

## tf2

- No changes

## tf2_eigen

```
* Updated to use ament_target_dependencies where possible. (ros#98 <ros2/geometry2#98>)
* Contributors: ivanpauno
```

## tf2_geometry_msgs

```
* Updated to use ament_target_dependencies where possible. (ros#98 <ros2/geometry2#98>)
```

## tf2_msgs

- No changes

## tf2_ros

```
* Updated to use node inteface pointers in the MessageFilter class. (ros#96 <ros2/geometry2#96>)
* Updated message_filter.h. (ros#91 <ros2/geometry2#91>)
* Contributors: Michael Jeronimo, Sagnik Basu
```

## tf2_sensor_msgs

- No changes
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.

2 participants