Skip to content

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

Merged
jacobperron merged 2 commits intomasterfrom
jacob/action_server_wait_set
Nov 21, 2018
Merged

[rcl_action] Implement wait set functions for action server#337
jacobperron merged 2 commits intomasterfrom
jacob/action_server_wait_set

Conversation

@jacobperron
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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