Skip to content

Commit 5395f9c

Browse files
fix linting problems
1 parent 9a8157d commit 5395f9c

File tree

2 files changed

+18
-17
lines changed

2 files changed

+18
-17
lines changed

nav2_dwb_controller/dwb_plugins/include/dwb_plugins/kinematic_parameters.hpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,11 +147,11 @@ class KinematicsHandler
147147

148148
inline KinematicParameters getKinematics()
149149
{
150-
KinematicParameters * ptr = kinematics_.load();
151-
if (ptr == nullptr) {
150+
KinematicParameters * ptr = kinematics_.load();
151+
if (ptr == nullptr) {
152152
throw std::runtime_error("Can't call KinematicsHandler::getKinematics().");
153-
}
154-
return *ptr;
153+
}
154+
return *ptr;
155155
}
156156

157157
void setSpeedLimit(const double & speed_limit, const bool & percentage);

nav2_dwb_controller/dwb_plugins/src/kinematic_parameters.cpp

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ KinematicsHandler::KinematicsHandler()
5555

5656
KinematicsHandler::~KinematicsHandler()
5757
{
58-
KinematicParameters* ptr = kinematics_.load();
58+
KinematicParameters * ptr = kinematics_.load();
5959
if (ptr != nullptr) {
6060
delete ptr;
6161
}
@@ -154,11 +154,11 @@ void KinematicsHandler::deactivate()
154154
void KinematicsHandler::setSpeedLimit(
155155
const double & speed_limit, const bool & percentage)
156156
{
157-
KinematicParameters* ptr = kinematics_.load();
158-
if (ptr == nullptr) {
159-
return; // Nothing to update
160-
}
161-
KinematicParameters kinematics(*ptr);
157+
KinematicParameters * ptr = kinematics_.load();
158+
if (ptr == nullptr) {
159+
return; // Nothing to update
160+
}
161+
KinematicParameters kinematics(*ptr);
162162

163163
if (speed_limit == nav2_costmap_2d::NO_SPEED_LIMIT) {
164164
// Restore default value
@@ -240,10 +240,10 @@ KinematicsHandler::updateParametersCallback(std::vector<rclcpp::Parameter> param
240240
{
241241
rcl_interfaces::msg::SetParametersResult result;
242242
KinematicParameters* ptr = kinematics_.load();
243-
if (ptr == nullptr) {
244-
return; // Nothing to update
245-
}
246-
KinematicParameters kinematics(*ptr);
243+
if (ptr == nullptr) {
244+
return; // Nothing to update
245+
}
246+
KinematicParameters kinematics(*ptr);
247247

248248
for (auto parameter : parameters) {
249249
const auto & param_type = parameter.get_type();
@@ -293,9 +293,10 @@ KinematicsHandler::updateParametersCallback(std::vector<rclcpp::Parameter> param
293293
update_kinematics(kinematics);
294294
}
295295

296-
void KinematicsHandler::update_kinematics(KinematicParameters kinematics) {
297-
KinematicParameters* new_kinematics = new KinematicParameters(kinematics);
298-
KinematicParameters* old_kinematics = kinematics_.exchange(new_kinematics);
296+
void KinematicsHandler::update_kinematics(KinematicParameters kinematics)
297+
{
298+
KinematicParameters * new_kinematics = new KinematicParameters(kinematics);
299+
KinematicParameters * old_kinematics = kinematics_.exchange(new_kinematics);
299300
if (old_kinematics != nullptr) {
300301
delete old_kinematics;
301302
}

0 commit comments

Comments
 (0)