Make get_name() return a const reference (#1952)

ros-controls · Dec 18, 2024 · e7d2fcc · e7d2fcc
1 parent b039baa
commit e7d2fcc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/hardware_interface/include/hardware_interface/loaned_command_interface.hpp b/hardware_interface/include/hardware_interface/loaned_command_interface.hpp
@@ -81,7 +81,7 @@ class LoanedCommandInterface
     }
   }
 
-  const std::string get_name() const { return command_interface_.get_name(); }
+  const std::string & get_name() const { return command_interface_.get_name(); }
 
   const std::string & get_interface_name() const { return command_interface_.get_interface_name(); }
 

diff --git a/hardware_interface/include/hardware_interface/loaned_state_interface.hpp b/hardware_interface/include/hardware_interface/loaned_state_interface.hpp
@@ -75,7 +75,7 @@ class LoanedStateInterface
     }
   }
 
-  const std::string get_name() const { return state_interface_.get_name(); }
+  const std::string & get_name() const { return state_interface_.get_name(); }
 
   const std::string & get_interface_name() const { return state_interface_.get_interface_name(); }
-Original file line number
+Diff line change
@@ Expand Up / @@ -81,7 +81,7 @@ class LoanedCommandInterface @@
         }
       }
-      const std::string get_name() const { return command_interface_.get_name(); }
+      const std::string & get_name() const { return command_interface_.get_name(); }
       const std::string & get_interface_name() const { return command_interface_.get_interface_name(); }
@@ Expand Down @@