fuse -> ROS 2: Clean up macro usage warnings (#280)

locusrobotics · Oct 13, 2022 · 7a453f6 · 7a453f6
1 parent e0dd00e
commit 7a453f6
Show file tree

Hide file tree

Showing 100 changed files with 121 additions and 121 deletions.
diff --git a/fuse_constraints/include/fuse_constraints/absolute_constraint.h b/fuse_constraints/include/fuse_constraints/absolute_constraint.h
@@ -72,7 +72,7 @@ template<class Variable>
 class AbsoluteConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(AbsoluteConstraint<Variable>);
+  FUSE_CONSTRAINT_DEFINITIONS(AbsoluteConstraint<Variable>)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/absolute_orientation_3d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/absolute_orientation_3d_stamped_constraint.h
@@ -65,7 +65,7 @@ namespace fuse_constraints
 class AbsoluteOrientation3DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(AbsoluteOrientation3DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(AbsoluteOrientation3DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/absolute_orientation_3d_stamped_euler_constraint.h b/fuse_constraints/include/fuse_constraints/absolute_orientation_3d_stamped_euler_constraint.h
@@ -67,7 +67,7 @@ namespace fuse_constraints
 class AbsoluteOrientation3DStampedEulerConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(AbsoluteOrientation3DStampedEulerConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS(AbsoluteOrientation3DStampedEulerConstraint)
 
   using Euler = fuse_variables::Orientation3DStamped::Euler;
 

diff --git a/fuse_constraints/include/fuse_constraints/absolute_pose_2d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/absolute_pose_2d_stamped_constraint.h
@@ -68,7 +68,7 @@ namespace fuse_constraints
 class AbsolutePose2DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(AbsolutePose2DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS(AbsolutePose2DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/absolute_pose_3d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/absolute_pose_3d_stamped_constraint.h
@@ -68,7 +68,7 @@ namespace fuse_constraints
 class AbsolutePose3DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(AbsolutePose3DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(AbsolutePose3DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/marginal_constraint.h b/fuse_constraints/include/fuse_constraints/marginal_constraint.h
@@ -72,7 +72,7 @@ namespace fuse_constraints
 class MarginalConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(MarginalConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS(MarginalConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/normal_delta_orientation_3d_cost_functor.h b/fuse_constraints/include/fuse_constraints/normal_delta_orientation_3d_cost_functor.h
@@ -69,7 +69,7 @@ namespace fuse_constraints
 class NormalDeltaOrientation3DCostFunctor
 {
 public:
-  FUSE_MAKE_ALIGNED_OPERATOR_NEW();
+  FUSE_MAKE_ALIGNED_OPERATOR_NEW()
 
   /**
    * @brief Construct a cost function instance

diff --git a/fuse_constraints/include/fuse_constraints/normal_delta_pose_3d_cost_functor.h b/fuse_constraints/include/fuse_constraints/normal_delta_pose_3d_cost_functor.h
@@ -68,7 +68,7 @@ namespace fuse_constraints
 class NormalDeltaPose3DCostFunctor
 {
 public:
-  FUSE_MAKE_ALIGNED_OPERATOR_NEW();
+  FUSE_MAKE_ALIGNED_OPERATOR_NEW()
 
   /**
    * @brief Constructor

diff --git a/fuse_constraints/include/fuse_constraints/normal_prior_orientation_3d_cost_functor.h b/fuse_constraints/include/fuse_constraints/normal_prior_orientation_3d_cost_functor.h
@@ -69,7 +69,7 @@ namespace fuse_constraints
 class NormalPriorOrientation3DCostFunctor
 {
 public:
-  FUSE_MAKE_ALIGNED_OPERATOR_NEW();
+  FUSE_MAKE_ALIGNED_OPERATOR_NEW()
 
   /**
    * @brief Construct a cost function instance

diff --git a/fuse_constraints/include/fuse_constraints/normal_prior_pose_3d_cost_functor.h b/fuse_constraints/include/fuse_constraints/normal_prior_pose_3d_cost_functor.h
@@ -68,7 +68,7 @@ namespace fuse_constraints
 class NormalPriorPose3DCostFunctor
 {
 public:
-  FUSE_MAKE_ALIGNED_OPERATOR_NEW();
+  FUSE_MAKE_ALIGNED_OPERATOR_NEW()
 
   /**
    * @brief Construct a cost function instance

diff --git a/fuse_constraints/include/fuse_constraints/relative_constraint.h b/fuse_constraints/include/fuse_constraints/relative_constraint.h
@@ -75,7 +75,7 @@ template<class Variable>
 class RelativeConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(RelativeConstraint<Variable>);
+  FUSE_CONSTRAINT_DEFINITIONS(RelativeConstraint<Variable>)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/relative_orientation_3d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/relative_orientation_3d_stamped_constraint.h
@@ -64,7 +64,7 @@ namespace fuse_constraints
 class RelativeOrientation3DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(RelativeOrientation3DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(RelativeOrientation3DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/relative_pose_2d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/relative_pose_2d_stamped_constraint.h
@@ -66,7 +66,7 @@ namespace fuse_constraints
 class RelativePose2DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(RelativePose2DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS(RelativePose2DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/include/fuse_constraints/relative_pose_3d_stamped_constraint.h b/fuse_constraints/include/fuse_constraints/relative_pose_3d_stamped_constraint.h
@@ -65,7 +65,7 @@ namespace fuse_constraints
 class RelativePose3DStampedConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(RelativePose3DStampedConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS_WITH_EIGEN(RelativePose3DStampedConstraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_constraints/test/test_marginalize_variables.cpp b/fuse_constraints/test/test_marginalize_variables.cpp
@@ -62,7 +62,7 @@
 class GenericVariable : public fuse_core::Variable
 {
 public:
-  FUSE_VARIABLE_DEFINITIONS(GenericVariable);
+  FUSE_VARIABLE_DEFINITIONS(GenericVariable)
 
   GenericVariable() :
     fuse_core::Variable(fuse_core::uuid::generate()),
@@ -110,7 +110,7 @@ BOOST_CLASS_EXPORT(GenericVariable);
 class GenericConstraint : public fuse_core::Constraint
 {
 public:
-  FUSE_CONSTRAINT_DEFINITIONS(GenericConstraint);
+  FUSE_CONSTRAINT_DEFINITIONS(GenericConstraint)
 
   GenericConstraint(std::initializer_list<fuse_core::UUID> variable_uuids) :
     Constraint("test", variable_uuids) {}
@@ -145,7 +145,7 @@ class GenericConstraint : public fuse_core::Constraint
 class FixedOrientation3DStamped : public fuse_variables::Orientation3DStamped
 {
 public:
-  FUSE_VARIABLE_DEFINITIONS(FixedOrientation3DStamped);
+  FUSE_VARIABLE_DEFINITIONS(FixedOrientation3DStamped)
 
   FixedOrientation3DStamped() = default;
 

diff --git a/fuse_core/include/fuse_core/async_motion_model.h b/fuse_core/include/fuse_core/async_motion_model.h
@@ -85,7 +85,7 @@ namespace fuse_core
 class AsyncMotionModel : public MotionModel
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(AsyncMotionModel);
+  FUSE_SMART_PTR_ALIASES_ONLY(AsyncMotionModel)
 
   /**
    * @brief Destructor

diff --git a/fuse_core/include/fuse_core/async_publisher.h b/fuse_core/include/fuse_core/async_publisher.h
@@ -63,7 +63,7 @@ namespace fuse_core
 class AsyncPublisher : public Publisher
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(AsyncPublisher);
+  FUSE_SMART_PTR_ALIASES_ONLY(AsyncPublisher)
 
   /**
    * @brief Destructor

diff --git a/fuse_core/include/fuse_core/async_sensor_model.h b/fuse_core/include/fuse_core/async_sensor_model.h
@@ -87,7 +87,7 @@ namespace fuse_core
 class AsyncSensorModel : public SensorModel
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(AsyncSensorModel);
+  FUSE_SMART_PTR_ALIASES_ONLY(AsyncSensorModel)
 
   /**
    * @brief Destructor

diff --git a/fuse_core/include/fuse_core/autodiff_local_parameterization.h b/fuse_core/include/fuse_core/autodiff_local_parameterization.h
@@ -77,7 +77,7 @@ template <typename PlusFunctor, typename MinusFunctor, int kGlobalSize, int kLoc
 class AutoDiffLocalParameterization : public LocalParameterization
 {
 public:
-  FUSE_SMART_PTR_DEFINITIONS(AutoDiffLocalParameterization<PlusFunctor, MinusFunctor, kGlobalSize, kLocalSize>);
+  FUSE_SMART_PTR_DEFINITIONS(AutoDiffLocalParameterization<PlusFunctor, MinusFunctor, kGlobalSize, kLocalSize>)
 
   /**
    * @brief Constructs new PlusFunctor and MinusFunctor instances

diff --git a/fuse_core/include/fuse_core/constraint.h b/fuse_core/include/fuse_core/constraint.h
@@ -61,7 +61,7 @@
  * class Derived : public Constraint
  * {
  * public:
- *   FUSE_CONSTRAINT_CLONE_DEFINITION(Derived);
+ *   FUSE_CONSTRAINT_CLONE_DEFINITION(Derived)
  *   // The rest of the derived constraint implementation
  * }
  * @endcode
@@ -80,7 +80,7 @@
  * class Derived : public Constraint
  * {
  * public:
- *   FUSE_CONSTRAINT_SERIALIZE_DEFINITION(Derived);
+ *   FUSE_CONSTRAINT_SERIALIZE_DEFINITION(Derived)
  *   // The rest of the derived constraint implementation
  * }
  * @endcode
@@ -113,7 +113,7 @@
  * class Derived : public Constraint
  * {
  * public:
- *   FUSE_CONSTRAINT_TYPE_DEFINITION(Derived);
+ *   FUSE_CONSTRAINT_TYPE_DEFINITION(Derived)
  *   // The rest of the derived constraint implementation
  * }
  * @endcode
@@ -139,7 +139,7 @@
  * class Derived : public Constraint
  * {
  * public:
- *   FUSE_CONSTRAINT_DEFINITIONS(Derived);
+ *   FUSE_CONSTRAINT_DEFINITIONS(Derived)
  *   // The rest of the derived constraint implementation
  * }
  * @endcode
@@ -159,7 +159,7 @@
  * class Derived : public Constraint
  * {
  * public:
- *   FUSE_CONSTRAINT_DEFINITIONS_WTIH_EIGEN(Derived);
+ *   FUSE_CONSTRAINT_DEFINITIONS_WTIH_EIGEN(Derived)
  *   // The rest of the derived constraint implementation
  * }
  * @endcode
@@ -193,7 +193,7 @@ namespace fuse_core
 class Constraint
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(Constraint);
+  FUSE_SMART_PTR_ALIASES_ONLY(Constraint)
 
   /**
    * @brief Default constructor

diff --git a/fuse_core/include/fuse_core/graph.h b/fuse_core/include/fuse_core/graph.h
@@ -65,7 +65,7 @@
  * class Derived : public Graph
  * {
  * public:
- *   FUSE_GRAPH_SERIALIZE_DEFINITION(Derived);
+ *   FUSE_GRAPH_SERIALIZE_DEFINITION(Derived)
  *   // The rest of the derived graph implementation
  * }
  * @endcode
@@ -98,7 +98,7 @@
  * class Derived : public Graph
  * {
  * public:
- *   FUSE_GRAPH_TYPE_DEFINITION(Derived);
+ *   FUSE_GRAPH_TYPE_DEFINITION(Derived)
  *   // The rest of the derived graph implementation
  * }
  * @endcode
@@ -124,7 +124,7 @@
 * class Derived : public Graph
 * {
 * public:
-*   FUSE_GRAPH_DEFINITIONS(Derived);
+*   FUSE_GRAPH_DEFINITIONS(Derived)
 *   // The rest of the derived graph implementation
 * }
 * @endcode
@@ -149,7 +149,7 @@ namespace fuse_core
 class Graph
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(Graph);
+  FUSE_SMART_PTR_ALIASES_ONLY(Graph)
 
   /**
    * @brief A range of fuse_ros::Constraint objects

diff --git a/fuse_core/include/fuse_core/local_parameterization.h b/fuse_core/include/fuse_core/local_parameterization.h
@@ -57,7 +57,7 @@ namespace fuse_core
 class LocalParameterization : public ceres::LocalParameterization
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(LocalParameterization);
+  FUSE_SMART_PTR_ALIASES_ONLY(LocalParameterization)
 
   /**
    * @brief Generalization of the subtraction operation

diff --git a/fuse_core/include/fuse_core/loss.h b/fuse_core/include/fuse_core/loss.h
@@ -53,7 +53,7 @@
  * class Derived : public Loss
  * {
  * public:
- *   FUSE_LOSS_CLONE_DEFINITION(Derived);
+ *   FUSE_LOSS_CLONE_DEFINITION(Derived)
  *   // The rest of the derived loss function implementation
  * }
  * @endcode
@@ -72,7 +72,7 @@
  * class Derived : public Loss
  * {
  * public:
- *   FUSE_LOSS_SERIALIZE_DEFINITION(Derived);
+ *   FUSE_LOSS_SERIALIZE_DEFINITION(Derived)
  *   // The rest of the derived loss function implementation
  * }
  * @endcode
@@ -105,7 +105,7 @@
  * class Derived : public Loss
  * {
  * public:
- *   FUSE_LOSS_TYPE_DEFINITION(Derived);
+ *   FUSE_LOSS_TYPE_DEFINITION(Derived)
  *   // The rest of the derived loss function implementation
  * }
  * @endcode
@@ -131,7 +131,7 @@
  * class Derived : public Loss
  * {
  * public:
- *   FUSE_LOSS_DEFINITIONS(Derived);
+ *   FUSE_LOSS_DEFINITIONS(Derived)
  *   // The rest of the derived loss function implementation
  * }
  * @endcode
@@ -169,7 +169,7 @@ namespace fuse_core
 class Loss
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(Loss);
+  FUSE_SMART_PTR_ALIASES_ONLY(Loss)
 
   static constexpr ceres::Ownership Ownership =
       ceres::Ownership::TAKE_OWNERSHIP;  //!< The ownership of the ceres::LossFunction* returned by lossFunction()

diff --git a/fuse_core/include/fuse_core/message_buffer.h b/fuse_core/include/fuse_core/message_buffer.h
@@ -61,7 +61,7 @@ template <typename Message>
 class MessageBuffer
 {
 public:
-  FUSE_SMART_PTR_DEFINITIONS(MessageBuffer<Message>);
+  FUSE_SMART_PTR_DEFINITIONS(MessageBuffer<Message>)
 
   /**
    * @brief A range of messages

diff --git a/fuse_core/include/fuse_core/motion_model.h b/fuse_core/include/fuse_core/motion_model.h
@@ -53,7 +53,7 @@ namespace fuse_core
 class MotionModel
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(MotionModel);
+  FUSE_SMART_PTR_ALIASES_ONLY(MotionModel)
 
   /**
    * @brief Destructor

diff --git a/fuse_core/include/fuse_core/publisher.h b/fuse_core/include/fuse_core/publisher.h
@@ -57,7 +57,7 @@ namespace fuse_core
 class Publisher
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(Publisher);
+  FUSE_SMART_PTR_ALIASES_ONLY(Publisher)
 
   /**
    * @brief Constructor

diff --git a/fuse_core/include/fuse_core/sensor_model.h b/fuse_core/include/fuse_core/sensor_model.h
@@ -68,7 +68,7 @@ using TransactionCallback = std::function<void(Transaction::SharedPtr transactio
 class SensorModel
 {
 public:
-  FUSE_SMART_PTR_ALIASES_ONLY(SensorModel);
+  FUSE_SMART_PTR_ALIASES_ONLY(SensorModel)
 
   /**
    * @brief Destructor

diff --git a/fuse_core/include/fuse_core/timestamp_manager.h b/fuse_core/include/fuse_core/timestamp_manager.h
@@ -65,7 +65,7 @@ namespace fuse_core
 class TimestampManager
 {
 public:
-  FUSE_SMART_PTR_DEFINITIONS(TimestampManager);
+  FUSE_SMART_PTR_DEFINITIONS(TimestampManager)
 
   /**
    * @brief Function that generates motion model constraints between the requested timestamps

diff --git a/fuse_core/include/fuse_core/transaction.h b/fuse_core/include/fuse_core/transaction.h
@@ -66,7 +66,7 @@ namespace fuse_core
 class Transaction
 {
 public:
-  FUSE_SMART_PTR_DEFINITIONS(Transaction);
+  FUSE_SMART_PTR_DEFINITIONS(Transaction)
 
   /**
    * @brief A range of Constraint::SharedPtr objects