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

[rcl_action] Implement wait set functions for action server #337

Merged
merged 2 commits into from
Nov 21, 2018

Conversation

jacobperron
Copy link
Member

Closes #306

Up to rcl_action:

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

@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 Nov 20, 2018
@jacobperron jacobperron mentioned this pull request Nov 20, 2018
10 tasks
@sloretz
Copy link
Contributor

sloretz commented Nov 21, 2018

It looks like there are new functions in action_server.c, but I don't see new declarations for them in action_server.h. Forget to commit that file?

@jacobperron
Copy link
Member Author

It looks like there are new functions in action_server.c, but I don't see new declarations for them in action_server.h. Forget to commit that file?

@sloretz These functions are declared in rcl_action/wait.h. Maybe it would be more clear if we either move wait set related declarations into action_client.h and action_server.h or create a new wait.c with their implementations?

@sloretz
Copy link
Contributor

sloretz commented Nov 21, 2018

Maybe it would be more clear if we either move wait set related declarations into action_client.h and action_server.h or create a new wait.c with their implementations?

Either of those would help, but I should have looked a little harder.

@jacobperron jacobperron merged commit f39ac3c into master Nov 21, 2018
@jacobperron jacobperron deleted the jacob/action_server_wait_set branch November 21, 2018 01:53
@jacobperron jacobperron removed the in review Waiting for review (Kanban column) label Nov 21, 2018
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