replace remaining typedefs by using declaration

I have no idea why these were not picked up by clang-tidy.

Author: v4hn

core/include/moveit/task_constructor/cost_queue.h

@@ -29,7 +29,7 @@ template <typename T, typename Compare = ValueOrPointeeLess<T>>
 class ordered
 {
 public:
-	typedef std::list<T> container_type;
+	using container_type = std::list<T>;
 	using value_type = typename container_type::value_type;
 	using size_type = typename container_type::size_type;
 	using difference_type = typename container_type::difference_type;

core/include/moveit/task_constructor/properties.h

@@ -75,7 +75,7 @@ class Property
 	         const boost::any& default_value);
 
 public:
-	typedef boost::typeindex::type_info type_info;
+	using type_info = boost::typeindex::type_info;
 
 	/// Construct a property holding a any value
 	Property();

core/include/moveit/task_constructor/stage_p.h

@@ -64,7 +64,7 @@ class StagePrivate
 
 public:
 	/// container type used to store children
-	typedef std::list<Stage::pointer> container_type;
+	using container_type = std::list<Stage::pointer>;
 	StagePrivate(Stage* me, const std::string& name);
 	virtual ~StagePrivate() = default;
 

core/include/moveit/task_constructor/storage.h

@@ -65,7 +65,6 @@ namespace task_constructor {
 class SolutionBase;
 MOVEIT_CLASS_FORWARD(InterfaceState)
 MOVEIT_CLASS_FORWARD(Interface)
-typedef std::weak_ptr<Interface> InterfaceWeakPtr;
 MOVEIT_CLASS_FORWARD(Stage)
 MOVEIT_CLASS_FORWARD(Introspection)
 

core/include/moveit/task_constructor/task.h

@@ -100,7 +100,7 @@ class Task : protected WrapperBase
 	void enableIntrospection(bool enable = true);
 	Introspection& introspection();
 
-	typedef std::function<void(const Task& t)> TaskCallback;
+	using TaskCallback = std::function<void(const Task& t)>;
 	using TaskCallbackList = std::list<TaskCallback>;
 	/// add function to be called after each top-level iteration
 	TaskCallbackList::const_iterator addTaskCallback(TaskCallback&& cb);

core/include/moveit/task_constructor/utils.h

@@ -117,4 +117,4 @@ class Flags
 	Int i;
 };
 
-#define DECLARE_FLAGS(Flags, Enum) typedef QFlags<Enum> Flags;
+#define DECLARE_FLAGS(Flags, Enum) using Flags = QFlags<Enum>;

core/src/container.cpp

@@ -304,7 +304,7 @@ struct SolutionCollector
 		solutions.emplace_back(std::make_pair(trace, cost));
 	}
 
-	typedef std::list<std::pair<SolutionSequence::container_type, double>> SolutionCostPairs;
+	using SolutionCostPairs = std::list<std::pair<SolutionSequence::container_type, double>>;
 	SolutionCostPairs solutions;
 	const size_t max_depth;
 };

core/src/properties.cpp

@@ -57,7 +57,7 @@ class PropertyTypeRegistry
 	Entry dummy_;
 
 	// map from type_info to corresponding converter functions
-	typedef std::map<std::type_index, Entry> RegistryMap;
+	using RegistryMap = std::map<std::type_index, Entry>;
 	RegistryMap types_;
 	// map from type names (type.name or ROS msg name) to entry in types_
 	using TypeNameMap = std::map<std::string, RegistryMap::iterator>;

core/src/stages/compute_ik.cpp

@@ -82,8 +82,8 @@ void ComputeIK::setTargetPose(const Eigen::Isometry3d& pose, const std::string&
 	setTargetPose(pose_msg);
 }
 
-// found IK solutions with a flag indicating validity
-typedef std::vector<std::vector<double>> IKSolutions;
+// found IK solutions
+using IKSolutions = std::vector<std::vector<double>>;
 
 namespace {
 

core/src/stages/fixed_cartesian_poses.cpp

@@ -44,7 +44,7 @@ namespace moveit {
 namespace task_constructor {
 namespace stages {
 
-typedef std::vector<geometry_msgs::PoseStamped> PosesList;
+using PosesList = std::vector<geometry_msgs::PoseStamped>;
 
 FixedCartesianPoses::FixedCartesianPoses(const std::string& name) : MonitoringGenerator(name) {
 	auto& p = properties();