clang-tidy fixes: use uint8_t enums

This commit is contained in:
Robert Haschke 2024-12-22 12:28:46 +01:00
parent 5067dbb432
commit 45ea86c163
11 changed files with 16 additions and 16 deletions

View File

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

View File

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

View File

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

View File

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

View File

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

View File

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

View File

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

View File

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

View File

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

View File

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

View File

@ -43,7 +43,7 @@ using IconMaskAndColor = QPair<QString, QColor>;
class Icon : public QVector<IconMaskAndColor> class Icon : public QVector<IconMaskAndColor>
{ {
public: public:
enum IconStyleOption enum IconStyleOption : uint8_t
{ {
NONE = 0, NONE = 0,
TINT = 1, TINT = 1,