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

Refactor server goal handle's try_canceling() function #603

Merged
merged 2 commits into from
Jan 8, 2019

Conversation

jacobperron
Copy link
Member

Makes use of rcl_action_goal_handle_is_cancelable() for one less rcl_action call.

Follow up from #593.
This PR does not require immediate attention.

Makes use of rcl_action_goal_handle_is_cancelable() for one less rcl_action call.
@jacobperron jacobperron added in progress Actively being worked on (Kanban column) in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Dec 7, 2018
@sloretz
Copy link
Contributor

sloretz commented Jan 8, 2019

CI (packages up to test_rclcpp)

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

@sloretz sloretz merged commit 5c92811 into master Jan 8, 2019
@sloretz sloretz deleted the refactor_server_goal_handle branch January 8, 2019 19:52
@sloretz sloretz removed the in review Waiting for review (Kanban column) label Jan 8, 2019
cho3 pushed a commit to cho3/rclcpp that referenced this pull request Jun 3, 2019
Makes use of rcl_action_goal_handle_is_cancelable() for one less rcl_action call.
nnmm pushed a commit to ApexAI/rclcpp that referenced this pull request Jul 9, 2022
* Remove deprecated CLI rules

Resolves ros2#500

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Remove tests for deprecated CLI arguments

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
DensoADAS pushed a commit to DensoADAS/rclcpp that referenced this pull request Aug 5, 2022
…ge (ros2#603)

* Mutex-protected writes and topic creation/removal in sqlite_storage

Signed-off-by: Adam Dabrowski <adam.dabrowski@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

2 participants