Skip to content

Conversation

@sung-goo-kim
Copy link
Contributor

Currently, the while loop exit even if only one of futureOne and futureTwo done.
While loop should exit when both futureOne and futureTwo are done.

@sung-goo-kim sung-goo-kim changed the title (RclJava) Fix Node.testPubUInt32MultipleNodes() error (RclJava) Fix NodeTest.testPubUInt32MultipleNodes() error Sep 17, 2020
@jacobperron
Copy link
Contributor

@sung-goo-kim Thanks for the fix!

@esteve
Copy link
Member

esteve commented Sep 30, 2020

@sung-goo-kim thanks!

@esteve esteve merged commit 720cbed into ros2-java:dashing Sep 30, 2020
@sung-goo-kim sung-goo-kim deleted the dev/tc-error branch October 5, 2020 02:51
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 9, 2020
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 10, 2020
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 10, 2020
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 10, 2020
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 10, 2020
jacobperron pushed a commit to osrf/ros2_java that referenced this pull request Nov 10, 2020
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.

3 participants