Skip to content

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