Skip to content

Commit 318bfe6

Browse files
Mauro PasserinoAlberto Soragna
authored andcommitted
Add logs on failed take response/request (#107)
1 parent b46090e commit 318bfe6

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

rclcpp/src/rclcpp/client.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,10 @@ ClientBase::take_type_erased_response(void * response_out, rmw_request_id_t & re
9393
&request_header_out,
9494
response_out);
9595
if (RCL_RET_CLIENT_TAKE_FAILED == ret) {
96+
RCLCPP_ERROR(
97+
rclcpp::get_logger("rclcpp"),
98+
"Error in take_type_erased_response: RCL_RET_CLIENT_TAKE_FAILED. "
99+
"Service name: %s", get_service_name());
96100
return false;
97101
} else if (RCL_RET_OK != ret) {
98102
rclcpp::exceptions::throw_from_rcl_error(ret);

rclcpp/src/rclcpp/service.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,10 @@ ServiceBase::take_type_erased_request(void * request_out, rmw_request_id_t & req
6161
&request_id_out,
6262
request_out);
6363
if (RCL_RET_SERVICE_TAKE_FAILED == ret) {
64+
RCLCPP_ERROR(
65+
rclcpp::get_logger("rclcpp"),
66+
"Error in take_type_erased_request: RCL_RET_SERVICE_TAKE_FAILED. "
67+
"Service name: %s", get_service_name());
6468
return false;
6569
} else if (RCL_RET_OK != ret) {
6670
rclcpp::exceptions::throw_from_rcl_error(ret);

0 commit comments

Comments
 (0)