From 89a82334195c644a7c70ecf79068379afd924840 Mon Sep 17 00:00:00 2001 From: "LIU, CHAUNCY" Date: Fri, 1 Mar 2024 00:42:04 -0500 Subject: [PATCH] rclc: fix format strings to match type. Correct the format string in rclc_executor_spin_some and rclc_timer_init_default Signed-off-by: Chauncy Liu --- rclc/src/rclc/executor.c | 14 +++++++------- rclc/src/rclc/timer.c | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/rclc/src/rclc/executor.c b/rclc/src/rclc/executor.c index b91aa783..bbd51642 100644 --- a/rclc/src/rclc/executor.c +++ b/rclc/src/rclc/executor.c @@ -1833,7 +1833,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( ROS_PACKAGE_NAME, - "Subscription added to wait_set_subscription[%ld]", + "Subscription added to wait_set_subscription[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_subscription); @@ -1849,7 +1849,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) &executor->handles[i].index); if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( - ROS_PACKAGE_NAME, "Timer added to wait_set_timers[%ld]", + ROS_PACKAGE_NAME, "Timer added to wait_set_timers[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_timer); @@ -1866,7 +1866,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) &executor->handles[i].index); if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( - ROS_PACKAGE_NAME, "Service added to wait_set_service[%ld]", + ROS_PACKAGE_NAME, "Service added to wait_set_service[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_service); @@ -1884,7 +1884,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) &executor->handles[i].index); if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( - ROS_PACKAGE_NAME, "Client added to wait_set_client[%ld]", + ROS_PACKAGE_NAME, "Client added to wait_set_client[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_client); @@ -1900,7 +1900,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) &executor->handles[i].index); if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( - ROS_PACKAGE_NAME, "Guard_condition added to wait_set_client[%ld]", + ROS_PACKAGE_NAME, "Guard_condition added to wait_set_client[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_guard_condition); @@ -1932,7 +1932,7 @@ rclc_executor_spin_some(rclc_executor_t * executor, const uint64_t timeout_ns) if (rc == RCL_RET_OK) { RCUTILS_LOG_DEBUG_NAMED( ROS_PACKAGE_NAME, - "Action server added to wait_set_action_servers[%ld]", + "Action server added to wait_set_action_servers[%d]", executor->handles[i].index); } else { PRINT_RCLC_ERROR(rclc_executor_spin_some, rcl_wait_set_add_action_server); @@ -1977,7 +1977,7 @@ rclc_executor_spin(rclc_executor_t * executor) rcl_ret_t ret = RCL_RET_OK; RCUTILS_LOG_DEBUG_NAMED( ROS_PACKAGE_NAME, - "INFO: rcl_wait timeout %ld ms", + "INFO: rcl_wait timeout %lld ms", ((executor->timeout_ns / 1000) / 1000)); while (true) { ret = rclc_executor_spin_some(executor, executor->timeout_ns); diff --git a/rclc/src/rclc/timer.c b/rclc/src/rclc/timer.c index f480f9e6..85591172 100644 --- a/rclc/src/rclc/timer.c +++ b/rclc/src/rclc/timer.c @@ -42,7 +42,7 @@ rclc_timer_init_default( if (rc != RCL_RET_OK) { PRINT_RCLC_ERROR(rclc_timer_init_default, rcl_timer_init); } else { - RCUTILS_LOG_INFO("Created a timer with period %ld ms.\n", timeout_ns / 1000000); + RCUTILS_LOG_INFO("Created a timer with period %lld ms.\n", timeout_ns / 1000000); } return rc; }