Use ! in place of not keyword for Pilz and CHOMP planners (#3217) (#3221

) Co-authored-by: JafarAbdi <[email protected]> (cherry picked from commit 549a2ee) Co-authored-by: Silvio Traversaro <[email protected]>
moveit · Jan 10, 2025 · 51de436 · 51de436
1 parent 9d5c520
commit 51de436
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/moveit_planners/chomp/chomp_motion_planner/src/chomp_planner.cpp b/moveit_planners/chomp/chomp_motion_planner/src/chomp_planner.cpp
@@ -260,7 +260,7 @@ bool ChompPlanner::solve(const planning_scene::PlanningSceneConstPtr& planning_s
   res.processing_time_[0] = std::chrono::duration<double>(std::chrono::system_clock::now() - start_time).count();
 
   // report planning failure if path has collisions
-  if (not optimizer->isCollisionFree())
+  if (!optimizer->isCollisionFree())
   {
     RCLCPP_ERROR(LOGGER, "Motion plan is invalid.");
     res.error_code_.val = moveit_msgs::msg::MoveItErrorCodes::INVALID_MOTION_PLAN;

diff --git a/...anners/pilz_industrial_motion_planner/include/joint_limits_copy/joint_limits_rosparam.hpp b/...anners/pilz_industrial_motion_planner/include/joint_limits_copy/joint_limits_rosparam.hpp
@@ -32,7 +32,7 @@ namespace
 template <typename T>
 void declare_parameter(const rclcpp::Node::SharedPtr& node, const std::string& name, T default_value)
 {
-  if (not node->has_parameter(name))
+  if (!node->has_parameter(name))
   {
     node->declare_parameter<T>(name, default_value);
   }