Rename return type

This should better reflect what this is meant for.
ros-controls · May 3, 2023 · 508e619 · 508e619
1 parent 140761d
commit 508e619
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/hardware_interface/include/hardware_interface/types/hardware_interface_return_values.hpp b/hardware_interface/include/hardware_interface/types/hardware_interface_return_values.hpp
@@ -23,7 +23,7 @@ enum class return_type : std::uint8_t
 {
   OK = 0,
   ERROR = 1,
-  INACTIVE = 2,
+  READ_ONLY = 2,
 };
 
 }  // namespace hardware_interface

diff --git a/hardware_interface/src/resource_manager.cpp b/hardware_interface/src/resource_manager.cpp
@@ -1088,7 +1088,7 @@ HardwareReadWriteStatus ResourceManager::read(
         read_write_status.failed_hardware_names.push_back(component.get_name());
         resource_storage_->remove_all_hardware_interfaces_from_available_list(component.get_name());
       }
-      else if (ret_val == return_type::INACTIVE)
+      else if (ret_val == return_type::READ_ONLY)
       {
         using lifecycle_msgs::msg::State;
         rclcpp_lifecycle::State state(
@@ -1130,7 +1130,7 @@ HardwareReadWriteStatus ResourceManager::write(
         read_write_status.failed_hardware_names.push_back(component.get_name());
         resource_storage_->remove_all_hardware_interfaces_from_available_list(component.get_name());
       }
-      else if (ret_val == return_type::INACTIVE)
+      else if (ret_val == return_type::READ_ONLY)
       {
         using lifecycle_msgs::msg::State;
         rclcpp_lifecycle::State state(