Skip to content

Commit 0cd5d7a

Browse files
rbonghibmagyar
authored andcommitted
Fix warning dynamic_reconfigure
1 parent 4edfa42 commit 0cd5d7a

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

diff_drive_controller/include/diff_drive_controller/diff_drive_controller.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,8 +234,8 @@ namespace diff_drive_controller{
234234

235235
/// Dynamic Reconfigure server
236236
typedef dynamic_reconfigure::Server<DiffDriveControllerConfig> ReconfigureServer;
237-
238237
std::shared_ptr<ReconfigureServer> dyn_reconf_server_;
238+
boost::recursive_mutex dyn_reconf_server_mutex_;
239239

240240
private:
241241
/**

diff_drive_controller/src/diff_drive_controller.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -374,8 +374,13 @@ namespace diff_drive_controller{
374374
config.publish_rate = publish_rate;
375375
config.enable_odom_tf = enable_odom_tf_;
376376

377-
dyn_reconf_server_ = std::make_shared<ReconfigureServer>(controller_nh);
377+
dyn_reconf_server_ = std::make_shared<ReconfigureServer>(dyn_reconf_server_mutex_, controller_nh);
378+
379+
// Update parameters
380+
dyn_reconf_server_mutex_.lock();
378381
dyn_reconf_server_->updateConfig(config);
382+
dyn_reconf_server_mutex_.unlock();
383+
379384
dyn_reconf_server_->setCallback(boost::bind(&DiffDriveController::reconfCallback, this, _1, _2));
380385

381386
return true;

0 commit comments

Comments
 (0)