Rename return type

This should better reflect what this is meant for.
ros-controls · May 15, 2023 · 2dffe5d · 2dffe5d
1 parent 23dd762
commit 2dffe5d
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
@@ -1079,7 +1079,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(State::PRIMARY_STATE_INACTIVE, lifecycle_state_names::INACTIVE);
@@ -1121,7 +1121,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(State::PRIMARY_STATE_INACTIVE, lifecycle_state_names::INACTIVE);