Use .hpp headers (moveit#641)

based on PR moveit/moveit2#3113

Author: CihatAltiparmak

.clang-tidy

@@ -17,7 +17,6 @@ Checks:          'performance-*,
                   readability-identifier-naming,
                   '
 HeaderFilterRegex: '.*/moveit/task_constructor/.*\.h'
-AnalyzeTemporaryDtors: false
 CheckOptions:
   - key:             llvm-namespace-comment.ShortNamespaceLines
     value:           '10'

capabilities/src/execute_task_solution_capability.cpp

@@ -36,13 +36,13 @@
 
 #include "execute_task_solution_capability.h"
 
-#include <moveit/moveit_cpp/moveit_cpp.h>
-#include <moveit/plan_execution/plan_execution.h>
-#include <moveit/trajectory_processing/trajectory_tools.h>
-#include <moveit/kinematic_constraints/utils.h>
-#include <moveit/move_group/capability_names.h>
-#include <moveit/robot_state/conversions.h>
-#include <moveit/utils/message_checks.h>
+#include <moveit/moveit_cpp/moveit_cpp.hpp>
+#include <moveit/plan_execution/plan_execution.hpp>
+#include <moveit/trajectory_processing/trajectory_tools.hpp>
+#include <moveit/kinematic_constraints/utils.hpp>
+#include <moveit/move_group/capability_names.hpp>
+#include <moveit/robot_state/conversions.hpp>
+#include <moveit/utils/message_checks.hpp>
 #include <fmt/format.h>
 
 namespace {

capabilities/src/execute_task_solution_capability.h

@@ -41,7 +41,7 @@
 
 #pragma once
 
-#include <moveit/move_group/move_group_capability.h>
+#include <moveit/move_group/move_group_capability.hpp>
 #include <rclcpp_action/rclcpp_action.hpp>
 
 #include <moveit_task_constructor_msgs/action/execute_task_solution.hpp>

core/include/moveit/task_constructor/container_p.h

@@ -39,7 +39,7 @@
 #pragma once
 
 #include <moveit/task_constructor/container.h>
-#include <moveit/macros/class_forward.h>
+#include <moveit/macros/class_forward.hpp>
 #include "stage_p.h"
 
 #include <boost/bimap.hpp>

core/include/moveit/task_constructor/introspection.h

@@ -38,7 +38,7 @@
 
 #pragma once
 
-#include <moveit/macros/class_forward.h>
+#include <moveit/macros/class_forward.hpp>
 #include <moveit_task_constructor_msgs/msg/task_description.hpp>
 #include <moveit_task_constructor_msgs/msg/task_statistics.hpp>
 #include <moveit_task_constructor_msgs/msg/solution.hpp>

core/include/moveit/task_constructor/marker_tools.h

@@ -2,7 +2,7 @@
 
 #include <rviz_marker_tools/marker_creation.h>
 #include <visualization_msgs/msg/marker.hpp>
-#include <moveit/macros/class_forward.h>
+#include <moveit/macros/class_forward.hpp>
 #include <functional>
 
 namespace planning_scene {

core/include/moveit/task_constructor/merge.h

@@ -37,9 +37,9 @@
 #pragma once
 
 #include <moveit/task_constructor/storage.h>
-#include <moveit/robot_model/robot_model.h>
-#include <moveit/robot_trajectory/robot_trajectory.h>
-#include <moveit/trajectory_processing/time_parameterization.h>
+#include <moveit/robot_model/robot_model.hpp>
+#include <moveit/robot_trajectory/robot_trajectory.hpp>
+#include <moveit/trajectory_processing/time_parameterization.hpp>
 
 namespace moveit {
 namespace task_constructor {

core/include/moveit/task_constructor/moveit_compat.h

@@ -38,8 +38,8 @@
 
 #pragma once
 
-#include <moveit/version.h>
-#include <moveit/macros/class_forward.h>
+#include <moveit/version.hpp>
+#include <moveit/macros/class_forward.hpp>
 
 #define MOVEIT_VERSION_GE(major, minor, patch)                                                \
 	(MOVEIT_VERSION_MAJOR * 1'000'000 + MOVEIT_VERSION_MINOR * 1'000 + MOVEIT_VERSION_PATCH >= \

core/include/moveit/task_constructor/solvers/cartesian_path.h

@@ -39,7 +39,7 @@
 #pragma once
 
 #include <moveit/task_constructor/solvers/planner_interface.h>
-#include <moveit/robot_state/cartesian_interpolator.h>
+#include <moveit/robot_state/cartesian_interpolator.hpp>
 
 namespace moveit {
 namespace task_constructor {

core/include/moveit/task_constructor/solvers/joint_interpolation.h

@@ -39,7 +39,7 @@
 #pragma once
 
 #include <moveit/task_constructor/solvers/planner_interface.h>
-#include <moveit/macros/class_forward.h>
+#include <moveit/macros/class_forward.hpp>
 
 namespace moveit {
 namespace task_constructor {