Initialize integers in test_parameter_event_handler.cpp to avoid undefined behavior (ros2#1609)

ivanpauno · web-flow · commit 51a4b2155e9e · 2021-03-31T10:58:37.000-03:00
Signed-off-by: Ivan Santiago Paunovic &lt;ivanpauno@ekumenlabs.com&gt;
diff --git a/rclcpp/test/rclcpp/test_parameter_event_handler.cpp b/rclcpp/test/rclcpp/test_parameter_event_handler.cpp
@@ -149,8 +149,8 @@ TEST_F(TestNode, RegisterParameterCallback)
 
 TEST_F(TestNode, SameParameterDifferentNode)
 {
-  int64_t int_param_node1;
-  int64_t int_param_node2;
+  int64_t int_param_node1{0};
+  int64_t int_param_node2{0};
 
   auto cb1 = [&int_param_node1](const rclcpp::Parameter & p) {
       int_param_node1 = p.get_value<int64_t>();

Original file line number	Diff line number	Diff line change
`@@ -149,8 +149,8 @@ TEST_F(TestNode, RegisterParameterCallback)`
`149`	`149`
`150`	`150`	`TEST_F(TestNode, SameParameterDifferentNode)`
`151`	`151`	`{`
`152`		`- int64_t int_param_node1;`
`153`		`- int64_t int_param_node2;`
	`152`	`+ int64_t int_param_node1{0};`
	`153`	`+ int64_t int_param_node2{0};`
`154`	`154`
`155`	`155`	`auto cb1 = [&int_param_node1](const rclcpp::Parameter & p) {`
`156`	`156`	`int_param_node1 = p.get_value<int64_t>();`