Skip to content

Commit

Permalink
revert(tier4_calibration_rviz_plugin): readability-identifier-naming (t…
Browse files Browse the repository at this point in the history
  • Loading branch information
h-ohta authored and boyali committed Oct 3, 2022
1 parent 5e8b325 commit 83c76d0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,13 +72,13 @@ void AccelBrakeMapCalibratorButtonPanel::onInitialize()
update_suggest_sub_ = raw_node->create_subscription<std_msgs::msg::Bool>(
topic_edit_->text().toStdString(), 10,
std::bind(
&AccelBrakeMapCalibratorButtonPanel::callback_update_suggest, this, std::placeholders::_1));
&AccelBrakeMapCalibratorButtonPanel::callbackUpdateSuggest, this, std::placeholders::_1));

client_ = raw_node->create_client<tier4_vehicle_msgs::srv::UpdateAccelBrakeMap>(
"/accel_brake_map_calibrator/update_map_dir");
}

void AccelBrakeMapCalibratorButtonPanel::callback_update_suggest(
void AccelBrakeMapCalibratorButtonPanel::callbackUpdateSuggest(
const std_msgs::msg::Bool::ConstSharedPtr msg)
{
if (after_calib_) {
Expand All @@ -96,20 +96,20 @@ void AccelBrakeMapCalibratorButtonPanel::callback_update_suggest(
calibration_button_->setEnabled(true);
}

void AccelBrakeMapCalibratorButtonPanel::edit_topic()
void AccelBrakeMapCalibratorButtonPanel::editTopic()
{
update_suggest_sub_.reset();
rclcpp::Node::SharedPtr raw_node =
this->getDisplayContext()->getRosNodeAbstraction().lock()->get_raw_node();
update_suggest_sub_ = raw_node->create_subscription<std_msgs::msg::Bool>(
topic_edit_->text().toStdString(), 10,
std::bind(
&AccelBrakeMapCalibratorButtonPanel::callback_update_suggest, this, std::placeholders::_1));
&AccelBrakeMapCalibratorButtonPanel::callbackUpdateSuggest, this, std::placeholders::_1));
calibration_button_->setText("Wait for subscribe topic");
calibration_button_->setEnabled(false);
}

void AccelBrakeMapCalibratorButtonPanel::push_calibration_button()
void AccelBrakeMapCalibratorButtonPanel::pushCalibrationButton()
{
// lock button
calibration_button_->setEnabled(false);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,11 @@ class AccelBrakeMapCalibratorButtonPanel : public rviz_common::Panel
public:
explicit AccelBrakeMapCalibratorButtonPanel(QWidget * parent = nullptr);
void onInitialize() override;
void callback_update_suggest(const std_msgs::msg::Bool::ConstSharedPtr msg);
void callbackUpdateSuggest(const std_msgs::msg::Bool::ConstSharedPtr msg);

public Q_SLOTS: // NOLINT for Qt
void edit_topic();
void push_calibration_button();
void editTopic();
void pushCalibrationButton();

protected:
rclcpp::Subscription<std_msgs::msg::Bool>::SharedPtr update_suggest_sub_;
Expand Down

0 comments on commit 83c76d0

Please sign in to comment.