From 2dffe5d30e6431a63c54fb0eb99f75514521e796 Mon Sep 17 00:00:00 2001 From: Felix Exner Date: Wed, 3 May 2023 17:04:47 +0200 Subject: [PATCH] Rename return type This should better reflect what this is meant for. --- .../types/hardware_interface_return_values.hpp | 2 +- hardware_interface/src/resource_manager.cpp | 4 ++-- 2 files changed, 3 insertions(+), 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 index dad907f6837..01af3e1f631 100644 --- 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 index e1f12106408..77a600852f7 100644 --- 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);