adding node name to the executor runtime_error (#1686)

* adding node name to the executor runtime_error Signed-off-by: Steve Macenski <[email protected]> * changing get_name() to get_fully_qualified_name() + linting Signed-off-by: Steve Macenski <[email protected]> * changing print format for tests Signed-off-by: Steve Macenski <[email protected]>
ros2 · Jun 6, 2021 · 7d8b269 · 7d8b269
1 parent 55d2f67
commit 7d8b269
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/rclcpp/src/rclcpp/executor.cpp b/rclcpp/src/rclcpp/executor.cpp
@@ -268,7 +268,9 @@ Executor::add_node(rclcpp::node_interfaces::NodeBaseInterface::SharedPtr node_pt
   // If the node already has an executor
   std::atomic_bool & has_executor = node_ptr->get_associated_with_executor_atomic();
   if (has_executor.exchange(true)) {
-    throw std::runtime_error("Node has already been added to an executor.");
+    throw std::runtime_error(
+            std::string("Node '") + node_ptr->get_fully_qualified_name() +
+            "' has already been added to an executor.");
   }
   std::lock_guard<std::mutex> guard{mutex_};
   for (auto & weak_group : node_ptr->get_callback_groups()) {