Fixed logging in example nodes

norro · norro · commit b7f992209ee7 · 2021-02-22T11:22:34.000+01:00
Signed-off-by: Nordmann Arne (CR/ADT3) &lt;arne.nordmann@de.bosch.com&gt;
diff --git a/system_modes_examples/src/drive_base.cpp b/system_modes_examples/src/drive_base.cpp
@@ -75,31 +75,31 @@ class DriveBase : public LifecycleNode
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_configure(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_configure()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_configure()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_activate(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_activate()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_activate()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_deactivate(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_deactivate()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_deactivate()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_cleanup(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_cleanup()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_cleanup()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
diff --git a/system_modes_examples/src/manipulator.cpp b/system_modes_examples/src/manipulator.cpp
@@ -71,31 +71,31 @@ class Manipulator : public LifecycleNode
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_configure(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_configure()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_configure()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_activate(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_activate()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_activate()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_deactivate(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_deactivate()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_deactivate()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }
 
   rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn
   on_cleanup(const rclcpp_lifecycle::State &)
   {
-    RCLCPP_INFO(get_logger(), "on_cleanup()", this->get_name());
+    RCLCPP_INFO(get_logger(), "%s on_cleanup()", this->get_name());
 
     return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;
   }

Original file line number	Diff line number	Diff line change
`@@ -75,31 +75,31 @@ class DriveBase : public LifecycleNode`
`75`	`75`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`76`	`76`	`on_configure(const rclcpp_lifecycle::State &)`
`77`	`77`	`{`
`78`		`- RCLCPP_INFO(get_logger(), "on_configure()", this->get_name());`
	`78`	`+ RCLCPP_INFO(get_logger(), "%s on_configure()", this->get_name());`
`79`	`79`
`80`	`80`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`84`	`84`	`on_activate(const rclcpp_lifecycle::State &)`
`85`	`85`	`{`
`86`		`- RCLCPP_INFO(get_logger(), "on_activate()", this->get_name());`
	`86`	`+ RCLCPP_INFO(get_logger(), "%s on_activate()", this->get_name());`
`87`	`87`
`88`	`88`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`92`	`92`	`on_deactivate(const rclcpp_lifecycle::State &)`
`93`	`93`	`{`
`94`		`- RCLCPP_INFO(get_logger(), "on_deactivate()", this->get_name());`
	`94`	`+ RCLCPP_INFO(get_logger(), "%s on_deactivate()", this->get_name());`
`95`	`95`
`96`	`96`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`100`	`100`	`on_cleanup(const rclcpp_lifecycle::State &)`
`101`	`101`	`{`
`102`		`- RCLCPP_INFO(get_logger(), "on_cleanup()", this->get_name());`
	`102`	`+ RCLCPP_INFO(get_logger(), "%s on_cleanup()", this->get_name());`
`103`	`103`
`104`	`104`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`105`	`105`	`}`
Original file line number	Diff line number	Diff line change
`@@ -71,31 +71,31 @@ class Manipulator : public LifecycleNode`
`71`	`71`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`72`	`72`	`on_configure(const rclcpp_lifecycle::State &)`
`73`	`73`	`{`
`74`		`- RCLCPP_INFO(get_logger(), "on_configure()", this->get_name());`
	`74`	`+ RCLCPP_INFO(get_logger(), "%s on_configure()", this->get_name());`
`75`	`75`
`76`	`76`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`77`	`77`	`}`
`78`	`78`
`79`	`79`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`80`	`80`	`on_activate(const rclcpp_lifecycle::State &)`
`81`	`81`	`{`
`82`		`- RCLCPP_INFO(get_logger(), "on_activate()", this->get_name());`
	`82`	`+ RCLCPP_INFO(get_logger(), "%s on_activate()", this->get_name());`
`83`	`83`
`84`	`84`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`88`	`88`	`on_deactivate(const rclcpp_lifecycle::State &)`
`89`	`89`	`{`
`90`		`- RCLCPP_INFO(get_logger(), "on_deactivate()", this->get_name());`
	`90`	`+ RCLCPP_INFO(get_logger(), "%s on_deactivate()", this->get_name());`
`91`	`91`
`92`	`92`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn`
`96`	`96`	`on_cleanup(const rclcpp_lifecycle::State &)`
`97`	`97`	`{`
`98`		`- RCLCPP_INFO(get_logger(), "on_cleanup()", this->get_name());`
	`98`	`+ RCLCPP_INFO(get_logger(), "%s on_cleanup()", this->get_name());`
`99`	`99`
`100`	`100`	`return rclcpp_lifecycle::node_interfaces::LifecycleNodeInterface::CallbackReturn::SUCCESS;`
`101`	`101`	`}`