Skip to content

Use node interface pointers in the MessageFilter class#96

Merged
tfoote merged 2 commits intoros2:ros2from
mjeronimo:ros2
Mar 13, 2019
Merged

Use node interface pointers in the MessageFilter class#96
tfoote merged 2 commits intoros2:ros2from
mjeronimo:ros2

Conversation

@mjeronimo
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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