Skip to content

Commit 446f60c

Browse files
author
Mauro Passerino
committed
Add logs on failed take response/request
1 parent 3c1fb0f commit 446f60c

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
@@ -80,6 +80,10 @@ ClientBase::take_type_erased_response(void * response_out, rmw_request_id_t & re
8080
&request_header_out,
8181
response_out);
8282
if (RCL_RET_CLIENT_TAKE_FAILED == ret) {
83+
RCLCPP_ERROR(
84+
rclcpp::get_logger("rclcpp"),
85+
"Error in take_type_erased_response: RCL_RET_CLIENT_TAKE_FAILED. "
86+
"Service name: %s", get_service_name());
8387
return false;
8488
} else if (RCL_RET_OK != ret) {
8589
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
@@ -43,6 +43,10 @@ ServiceBase::take_type_erased_request(void * request_out, rmw_request_id_t & req
4343
&request_id_out,
4444
request_out);
4545
if (RCL_RET_SERVICE_TAKE_FAILED == ret) {
46+
RCLCPP_ERROR(
47+
rclcpp::get_logger("rclcpp"),
48+
"Error in take_type_erased_request: RCL_RET_SERVICE_TAKE_FAILED. "
49+
"Service name: %s", get_service_name());
4650
return false;
4751
} else if (RCL_RET_OK != ret) {
4852
rclcpp::exceptions::throw_from_rcl_error(ret);

0 commit comments

Comments
 (0)