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

waitset -> wait_set #408

Merged
merged 4 commits into from
Nov 27, 2017
Merged

waitset -> wait_set #408

merged 4 commits into from
Nov 27, 2017

Conversation

mikaelarguedas
Copy link
Member

connects to ros2/rmw#131

@mikaelarguedas mikaelarguedas added the in progress Actively being worked on (Kanban column) label Nov 22, 2017
@mikaelarguedas mikaelarguedas merged commit 7bbf5f6 into master Nov 27, 2017
@mikaelarguedas mikaelarguedas removed the in progress Actively being worked on (Kanban column) label Nov 27, 2017
@mikaelarguedas mikaelarguedas deleted the wait_set_two_words branch November 27, 2017 21:30
nnmm pushed a commit to ApexAI/rclcpp that referenced this pull request Jul 9, 2022
* implement rcl_wait and rcl_take for rcl_event_t

Signed-off-by: Miaofei <[email protected]>

* address feedback regarding formatting issues

Signed-off-by: Miaofei <[email protected]>

* Remove dependency on sleep in test_events

Signed-off-by: Ross Desmond <[email protected]>

* update total max wait time for setting up publisher and subscriber to 10 seconds

Signed-off-by: Miaofei <[email protected]>

* Fix test_events for rmw_connext

Signed-off-by: Ross Desmond <[email protected]>

* Refactor out timed loop from tests to function

Signed-off-by: Miaofei <[email protected]>

* address additional feedback from pull request

Signed-off-by: Miaofei <[email protected]>

* update comment regarding difference between connext and opensplice

Signed-off-by: Miaofei <[email protected]>

* fix uncrustify issues

Signed-off-by: Miaofei <[email protected]>

* update test_events for compatibility with API changes

Signed-off-by: Miaofei <[email protected]>

* temporarily disable test_events for macOS

Signed-off-by: Miaofei <[email protected]>
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