fix: lint error handling

Author: Woojin-Crive

include/dynamixel_hardware_interface/dynamixel_hardware_interface.hpp

@@ -261,7 +261,7 @@ class DynamixelHardware : public
    * @param type_filter The type of items to initialize ("controller" or "dxl" or "sensor").
    * @return True if initialization was successful, false otherwise.
    */
-  bool initItems(const std::string& type_filter);
+  bool initItems(const std::string & type_filter);
 
   /**
    * @brief Helper function to retry writing an item to a Dynamixel device.
@@ -270,7 +270,7 @@ class DynamixelHardware : public
    * @param value The value to write.
    * @return True if write was successful, false if timeout occurred.
    */
-  bool retryWriteItem(uint8_t id, const std::string& item_name, uint32_t value);
+  bool retryWriteItem(uint8_t id, const std::string & item_name, uint32_t value);
 
   /**
    * @brief Initializes Dynamixel items.

src/dynamixel_hardware_interface.cpp

@@ -117,7 +117,7 @@ hardware_interface::CallbackReturn DynamixelHardware::on_init(
   int trying_cnt = 60;
   int cnt = 0;
 
-  if(controller_id_.size() > 0 ) {
+  if (controller_id_.size() > 0) {
     while (trying_connect) {
       std::vector<uint8_t> id_arr;
       for (auto controller : controller_id_) {
@@ -614,24 +614,25 @@ bool DynamixelHardware::CommReset()
   return false;
 }
 
-bool DynamixelHardware::retryWriteItem(uint8_t id, const std::string& item_name, uint32_t value) {
+bool DynamixelHardware::retryWriteItem(uint8_t id, const std::string & item_name, uint32_t value)
+{
   rclcpp::Time start_time = this->now();
   rclcpp::Duration error_duration(0, 0);
-  
+
   while (true) {
     if (dxl_comm_->WriteItem(id, item_name, value) == DxlError::OK) {
       RCLCPP_INFO_STREAM(
         logger_,
         "[ID:" << std::to_string(id) << "] item_name:" << item_name.c_str() << "\tdata:" << value);
       return true;
     }
-    
+
     error_duration = this->now() - start_time;
     if (error_duration.seconds() * 1000 >= err_timeout_ms_) {
       RCLCPP_ERROR_STREAM(
         logger_,
-        "[ID:" << std::to_string(id) << "] Write Item error (Timeout: " << 
-        error_duration.seconds() * 1000 << "ms/" << err_timeout_ms_ << "ms)");
+        "[ID:" << std::to_string(id) << "] Write Item error (Timeout: " <<
+          error_duration.seconds() * 1000 << "ms/" << err_timeout_ms_ << "ms)");
       return false;
     }
     RCLCPP_WARN_STREAM(
@@ -640,14 +641,15 @@ bool DynamixelHardware::retryWriteItem(uint8_t id, const std::string& item_name,
   }
 }
 
-bool DynamixelHardware::initItems(const std::string& type_filter) {
+bool DynamixelHardware::initItems(const std::string & type_filter)
+{
   RCLCPP_INFO_STREAM(logger_, "$$$$$ Init Items for type: " << type_filter);
   for (const hardware_interface::ComponentInfo & gpio : info_.gpios) {
     if (gpio.parameters.at("type") != type_filter) {
       continue;
     }
     uint8_t id = static_cast<uint8_t>(stoi(gpio.parameters.at("ID")));
-    
+
     // First write items containing "Limit"
     for (auto it : gpio.parameters) {
       if (it.first != "ID" && it.first != "type" && it.first.find("Limit") != std::string::npos) {