Fix more clang warnings

KanRobotics · Jan 8, 2023 · 499c16b · 499c16b
1 parent 27e8b65
commit 499c16b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/moveit_ros/hybrid_planning/test/hybrid_planning_demo_node.cpp b/moveit_ros/hybrid_planning/test/hybrid_planning_demo_node.cpp
@@ -265,7 +265,7 @@ class HybridPlanningDemo
     send_goal_options.feedback_callback =
         [](const rclcpp_action::ClientGoalHandle<moveit_msgs::action::HybridPlanner>::SharedPtr& /*unused*/,
            const std::shared_ptr<const moveit_msgs::action::HybridPlanner::Feedback>& feedback) {
-          RCLCPP_INFO(LOGGER, feedback->feedback.c_str());
+          RCLCPP_INFO_STREAM(LOGGER, feedback->feedback);
         };
 
     RCLCPP_INFO(LOGGER, "Sending hybrid planning goal");

diff --git a/moveit_ros/moveit_servo/src/servo_calcs.cpp b/moveit_ros/moveit_servo/src/servo_calcs.cpp
@@ -61,8 +61,6 @@ namespace moveit_servo
 {
 namespace
 {
-constexpr char CONDITION_TOPIC[] = "~/condition";
-
 static const rclcpp::Logger LOGGER = rclcpp::get_logger("moveit_servo.servo_calcs");
 constexpr auto ROS_LOG_THROTTLE_PERIOD = std::chrono::milliseconds(3000).count();
 static constexpr double STOPPED_VELOCITY_EPS = 1e-4;  // rad/s