fix doc comments and fix to move

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>

Author: Koichi98

common/autoware_agnocast_wrapper/include/autoware/agnocast_wrapper/node.hpp

@@ -420,7 +420,7 @@ class Node
   }
 };
 
-/// @brief Get the underlying rclcpp::Node from a node that inherits agnocast_wrapper::Node.
+/// @brief Get the underlying rclcpp::Node from an agnocast_wrapper::Node.
 /// @throws std::runtime_error if the node is in agnocast mode (check !use_agnocast() first)
 template <typename T>
 std::shared_ptr<rclcpp::Node> to_rclcpp_node(const std::shared_ptr<T> & node)
@@ -588,7 +588,7 @@ class Node
   rclcpp::node_interfaces::OnSetParametersCallbackHandle::SharedPtr add_on_set_parameters_callback(
     OnSetParametersCallbackType callback)
   {
-    return node_->add_on_set_parameters_callback(callback);
+    return node_->add_on_set_parameters_callback(std::move(callback));
   }
   void remove_on_set_parameters_callback(
     const rclcpp::node_interfaces::OnSetParametersCallbackHandle * const handler)
@@ -745,7 +745,7 @@ class Node
   std::shared_ptr<rclcpp::Node> node_;
 };
 
-/// @brief Get the underlying rclcpp::Node from a node that inherits agnocast_wrapper::Node.
+/// @brief Get the underlying rclcpp::Node from an agnocast_wrapper::Node.
 template <typename T>
 std::shared_ptr<rclcpp::Node> to_rclcpp_node(const std::shared_ptr<T> & node)
 {

common/autoware_agnocast_wrapper/src/node.cpp

@@ -182,7 +182,7 @@ rcl_interfaces::msg::ListParametersResult Node::list_parameters(
 rclcpp::node_interfaces::OnSetParametersCallbackHandle::SharedPtr
 Node::add_on_set_parameters_callback(OnSetParametersCallbackType callback)
 {
-  return visit_node([&](auto & n) { return n->add_on_set_parameters_callback(callback); });
+  return visit_node([&](auto & n) { return n->add_on_set_parameters_callback(std::move(callback)); });
 }
 
 void Node::remove_on_set_parameters_callback(