Fix platform-dependent warning in controller_interface_base.cpp using fmt::format (backport #2880)
#2882
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR updates a
RCLCPP_WARN_EXPRESSIONcall incontroller_interface_base.cppto usefmt::formatinstead of printf-style formatting.Changes
%ld/%dformat specifiers withfmt::formatfor platform-independent logging.ctrl_itf_params_.update_rateis updated toachievable_hzonly if the requested update rate is not achievable.Motivation
Printf-style format specifiers can cause warnings or undefined behavior depending on the platform (e.g.,
longvsint64_t). Usingfmt::formatensures type safety and platform independence while keeping the original functionality.This is an automatic backport of pull request #2880 done by [Mergify](https://mergify.com).