Fix bug in navsat_transform (#673)

kribe48 · web-flow · commit 46e035ff4f43 · 2021-06-11T08:57:50.000+01:00
When using the parameter "broadcast_cartesian_transform" as intended, the "broadcast_utm_transform" is declared twice.
diff --git a/src/navsat_transform.cpp b/src/navsat_transform.cpp
@@ -116,7 +116,7 @@ NavSatTransform::NavSatTransform(const rclcpp::NodeOptions & options)
       "Please use 'broadcast_cartesian_transform' instead.");
   } else {
     broadcast_cartesian_transform_ =
-      this->declare_parameter("broadcast_utm_transform", broadcast_cartesian_transform_);
+      this->declare_parameter("broadcast_cartesian_transform", broadcast_cartesian_transform_);
   }
 
   broadcast_cartesian_transform_as_parent_frame_ =

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ NavSatTransform::NavSatTransform(const rclcpp::NodeOptions & options)`
`116`	`116`	`"Please use 'broadcast_cartesian_transform' instead.");`
`117`	`117`	`} else {`
`118`	`118`	`broadcast_cartesian_transform_ =`
`119`		`- this->declare_parameter("broadcast_utm_transform", broadcast_cartesian_transform_);`
	`119`	`+ this->declare_parameter("broadcast_cartesian_transform", broadcast_cartesian_transform_);`
`120`	`120`	`}`
`121`	`121`
`122`	`122`	`broadcast_cartesian_transform_as_parent_frame_ =`