clang-tidy fixes: use uint8_t enums

Author: rhaschke

core/include/moveit/task_constructor/cost_terms.h

@@ -70,7 +70,7 @@ class CostTerm
 class TrajectoryCostTerm : public CostTerm
 {
 public:
-	enum class Mode
+	enum class Mode : uint8_t
 	{
 		AUTO /* TRAJECTORY, or START_INTERFACE if no trajectory is given */,
 		START_INTERFACE,

core/include/moveit/task_constructor/stage.h

@@ -72,7 +72,7 @@ MOVEIT_CLASS_FORWARD(RobotTrajectory);
 namespace moveit {
 namespace task_constructor {
 
-enum InterfaceFlag
+enum InterfaceFlag : uint8_t
 {
 	READS_START = 0x01,
 	READS_END = 0x02,
@@ -155,7 +155,7 @@ class Stage
 	 *
 	 * INTERFACE takes precedence over PARENT.
 	 */
-	enum PropertyInitializerSource
+	enum PropertyInitializerSource : uint8_t
 	{  // TODO: move to property.cpp
 		DEFAULT = 0,
 		MANUAL = 1,
@@ -293,7 +293,7 @@ class PropagatingEitherWay : public ComputeBase
 	PRIVATE_CLASS(PropagatingEitherWay)
 	PropagatingEitherWay(const std::string& name = "propagating either way");
 
-	enum Direction
+	enum Direction : uint8_t
 	{
 		AUTO = 0x00,  // auto-derive direction from context
 		FORWARD = 0x01,  // propagate forward only

core/include/moveit/task_constructor/stages/connect.h

@@ -67,7 +67,7 @@ class Connect : public Connecting
 	bool compatible(const InterfaceState& from_state, const InterfaceState& to_state) const override;
 
 public:
-	enum MergeMode
+	enum MergeMode : uint8_t
 	{
 		SEQUENTIAL = 0,
 		WAYPOINTS = 1

core/include/moveit/task_constructor/stages/generate_random_pose.h

@@ -57,7 +57,7 @@ class GenerateRandomPose : public GeneratePose
 	/** Function signature for pose dimension samplers.
 	 * The input parameter is the seed, the returned value is the sampling result. */
 	using PoseDimensionSampler = std::function<double(double)>;
-	enum PoseDimension
+	enum PoseDimension : uint8_t
 	{
 		X,
 		Y,

core/include/moveit/task_constructor/storage.h

@@ -80,7 +80,7 @@ class InterfaceState
 	friend class ContainerBasePrivate;  // allow setting priority_ for pruning
 
 public:
-	enum Status
+	enum Status : uint8_t
 	{
 		ENABLED,  // state is actively considered during planning
 		ARMED,  // disabled state in a Connecting interface that will become re-enabled with a new opposite state
@@ -197,12 +197,12 @@ class Interface : public ordered<InterfaceState*>
 		const InterfaceState* operator->() const noexcept { return base_type::const_iterator::operator*(); }
 	};
 
-	enum Direction
+	enum Direction : uint8_t
 	{
 		FORWARD,
 		BACKWARD,
 	};
-	enum Update
+	enum Update : uint8_t
 	{
 		STATUS = 1 << 0,
 		PRIORITY = 1 << 1,

core/test/test_container.cpp

@@ -15,7 +15,7 @@
 
 using namespace moveit::task_constructor;
 
-enum StageType
+enum StageType : uint8_t
 {
 	GEN,
 	FW,

rviz_marker_tools/include/rviz_marker_tools/marker_creation.h

@@ -11,7 +11,7 @@ class Geometry;
 
 namespace rviz_marker_tools {
 
-enum Color
+enum Color : uint8_t
 {
 	BLACK = 0,
 	BROWN = 1,

visualization/motion_planning_tasks/src/remote_task_model.cpp

@@ -54,7 +54,7 @@ using namespace moveit::task_constructor;
 
 namespace moveit_rviz_plugin {
 
-enum NodeFlag
+enum NodeFlag : uint8_t
 {
 	WAS_VISITED = 0x01,  // indicate that model should emit change notifications
 	NAME_CHANGED = 0x02,  // indicate that name was manually changed

visualization/motion_planning_tasks/src/task_list_model.h

@@ -77,7 +77,7 @@ class BaseTaskModel : public QAbstractItemModel
 	rviz::DisplayContext* display_context_;
 
 public:
-	enum TaskModelFlag
+	enum TaskModelFlag : uint8_t
 	{
 		LOCAL_MODEL = 0x01,
 		IS_DESTROYED = 0x02,

visualization/motion_planning_tasks/src/task_panel.h

@@ -118,7 +118,7 @@ class TaskView : public SubPanel
 
 protected:
 	// configuration settings
-	enum TaskExpand
+	enum TaskExpand : uint8_t
 	{
 		EXPAND_TOP = 1,
 		EXPAND_ALL,
@@ -130,7 +130,7 @@ class TaskView : public SubPanel
 	rviz::BoolProperty* show_time_column;
 
 public:
-	enum OldTaskHandling
+	enum OldTaskHandling : uint8_t
 	{
 		OLD_TASK_KEEP = 1,
 		OLD_TASK_REPLACE,