Skip to content

Commit e8a13c8

Browse files
committed
precommit checks
Signed-off-by: Dhruv Patel <[email protected]>
1 parent fa8ec65 commit e8a13c8

File tree

1 file changed

+22
-19
lines changed

1 file changed

+22
-19
lines changed

nav2_ros_common/include/nav2_ros_common/node_utils.hpp

Lines changed: 22 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -256,17 +256,18 @@ inline ParamType declare_or_get_parameter(
256256
}
257257

258258
auto return_value = param_interface
259-
->declare_parameter(parameter_name, rclcpp::ParameterValue{default_value},
260-
parameter_descriptor)
259+
->declare_parameter(
260+
parameter_name, rclcpp::ParameterValue{default_value},
261+
parameter_descriptor)
261262
.get<ParamType>();
262263

263264
const bool no_param_override = param_interface->get_parameter_overrides().find(parameter_name) ==
264265
param_interface->get_parameter_overrides().end();
265266
if (no_param_override) {
266267
if (warn_if_no_override) {
267268
RCLCPP_WARN_STREAM(
268-
logger,
269-
"Failed to get param " << parameter_name << " from overrides, using default value.");
269+
logger,
270+
"Failed to get param " << parameter_name << " from overrides, using default value.");
270271
}
271272
if (strict_param_loading) {
272273
std::string description = "Parameter " + parameter_name +
@@ -305,7 +306,8 @@ inline ParamType declare_or_get_parameter(
305306
declare_parameter_if_not_declared(node, "strict_param_loading", rclcpp::ParameterValue(false));
306307
bool strict_param_loading{false};
307308
node->get_parameter("strict_param_loading", strict_param_loading);
308-
return declare_or_get_parameter(node->get_logger(), node->get_node_parameters_interface(),
309+
return declare_or_get_parameter(
310+
node->get_logger(), node->get_node_parameters_interface(),
309311
parameter_name, default_value, warn_if_no_override, strict_param_loading, parameter_descriptor);
310312
}
311313

@@ -352,21 +354,21 @@ inline void setSoftRealTimePriority()
352354
thread_precedence_policy_data_t policy;
353355
policy.importance = 63; // 0–63, higher = more priority
354356
kern_return_t result = thread_policy_set(
355-
thread,
356-
THREAD_PRECEDENCE_POLICY,
357-
(thread_policy_t)&policy,
358-
THREAD_PRECEDENCE_POLICY_COUNT);
357+
thread,
358+
THREAD_PRECEDENCE_POLICY,
359+
(thread_policy_t)&policy,
360+
THREAD_PRECEDENCE_POLICY_COUNT);
359361

360362
if (result != KERN_SUCCESS) {
361-
// Construct the message once with the full context
362-
std::string errmsg =
363-
"Failed to set THREAD_PRECEDENCE_POLICY on macOS. "
364-
"Thread priority remains at default. "
365-
"Mach Error Code: ";
363+
// Construct the message once with the full context
364+
std::string errmsg =
365+
"Failed to set THREAD_PRECEDENCE_POLICY on macOS. "
366+
"Thread priority remains at default. "
367+
"Mach Error Code: ";
366368

367-
// Append the numerical result code and throw
368-
throw std::runtime_error(errmsg + std::to_string(result));
369-
}
369+
// Append the numerical result code and throw
370+
throw std::runtime_error(errmsg + std::to_string(result));
371+
}
370372
#else
371373
// Linux: True real-time scheduling (requires privileges)
372374
sched_param sch;
@@ -418,8 +420,9 @@ inline void replaceOrAddArgument(
418420
std::vector<std::string> & arguments, const std::string & option,
419421
const std::string & arg_name, const std::string & new_argument)
420422
{
421-
auto argument = std::find_if(arguments.begin(), arguments.end(),
422-
[arg_name](const std::string & value){return value.find(arg_name) != std::string::npos;});
423+
auto argument = std::find_if(
424+
arguments.begin(), arguments.end(),
425+
[arg_name](const std::string & value) {return value.find(arg_name) != std::string::npos;});
423426
if (argument != arguments.end()) {
424427
*argument = new_argument;
425428
} else {

0 commit comments

Comments
 (0)