Skip to content

Commit

Permalink
roscpp ActionClient subscription queue size should be consistent with…
Browse files Browse the repository at this point in the history
… rospy. (#162)

* roscpp ActionClient subscription queue size should be more than 1, and also consistent with rospy.
* expand test duration for test_exercise_simple_clients.
* Revert "expand test duration for test_exercise_simple_clients."
* expand test duration and timeout for test_exercise_simple_clients.
* Revert " expand test duration and timeout for test_exercise_simple_clients."
  • Loading branch information
fujitatomoya authored Apr 23, 2020
1 parent 9beedb8 commit 8ac7539
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions actionlib/include/actionlib/client/action_client.h
Original file line number Diff line number Diff line change
Expand Up @@ -221,9 +221,9 @@ class ActionClient
int pub_queue_size;
int sub_queue_size;
n_.param("actionlib_client_pub_queue_size", pub_queue_size, 10);
n_.param("actionlib_client_sub_queue_size", sub_queue_size, 1);
n_.param("actionlib_client_sub_queue_size", sub_queue_size, -1);
if (pub_queue_size < 0) {pub_queue_size = 10;}
if (sub_queue_size < 0) {sub_queue_size = 1;}
if (sub_queue_size < 0) {sub_queue_size = 0;}

status_sub_ = queue_subscribe("status", static_cast<uint32_t>(sub_queue_size),
&ActionClientT::statusCb, this, queue);
Expand Down

0 comments on commit 8ac7539

Please sign in to comment.