Make getClass() a const method

include/core/common/factory.h

@@ -244,7 +244,7 @@ class Factory
         bool registerCreator(const std::string& _type, CreatorCallback createFn);
         bool unregisterCreator(const std::string& _type);
         TypeBasePtr create(const std::string& _type, TypeInput... _input);
-        std::string getClass();
+        std::string getClass() const;
 
     private:
         CallbackMap callbacks_;
@@ -301,7 +301,7 @@ inline Factory<TypeBase, TypeInput...>& Factory<TypeBase, TypeInput...>::get()
 }
 
 template<class TypeBase, typename... TypeInput>
-inline std::string Factory<TypeBase, TypeInput...>::getClass()
+inline std::string Factory<TypeBase, TypeInput...>::getClass() const
 {
     return "Factory<class TypeBase>";
 }
@@ -319,7 +319,7 @@ class LandmarkBase;
 typedef Factory<LandmarkBase,
         const YAML::Node&>  FactoryLandmark;
 template<>
-inline std::string FactoryLandmark::getClass()
+inline std::string FactoryLandmark::getClass() const
 {
     return "FactoryLandmark";
 }

include/core/processor/factory_processor.h

@@ -170,7 +170,7 @@ struct ParamsProcessorBase;
 typedef Factory<ParamsProcessorBase,
         const std::string&> FactoryParamsProcessor;
 template<>
-inline std::string FactoryParamsProcessor::getClass()
+inline std::string FactoryParamsProcessor::getClass() const
 {
     return "FactoryParamsProcessor";
 }
@@ -180,7 +180,7 @@ typedef Factory<ProcessorBase,
         const std::string&,
         const ParamsProcessorBasePtr> FactoryProcessor;
 template<>
-inline std::string FactoryProcessor::getClass()
+inline std::string FactoryProcessor::getClass() const
 {
   return "FactoryProcessor";
 }
@@ -193,7 +193,7 @@ typedef Factory<ProcessorBase,
         const std::string&,
         const ParamsServer&> AutoConfFactoryProcessor;
 template<>
-inline std::string AutoConfFactoryProcessor::getClass()
+inline std::string AutoConfFactoryProcessor::getClass() const
 {
     return "AutoConfFactoryProcessor";
 }

include/core/sensor/factory_sensor.h

@@ -212,9 +212,9 @@ namespace wolf
 // ParamsSensor factory
 struct ParamsSensorBase;
 typedef Factory<ParamsSensorBase,
-        const std::string&> FactoryParamsSensor;
+                const std::string&> FactoryParamsSensor;
 template<>
-inline std::string FactoryParamsSensor::getClass()
+inline std::string FactoryParamsSensor::getClass() const
 {
     return "FactoryParamsSensor";
 }
@@ -222,10 +222,11 @@ inline std::string FactoryParamsSensor::getClass()
 // Sensor factory
 typedef Factory<SensorBase,
                 const std::string&,
-                const Eigen::VectorXd&, const ParamsSensorBasePtr> FactorySensor;
+                const Eigen::VectorXd&,
+                const ParamsSensorBasePtr> FactorySensor;
 
 template<>
-inline std::string FactorySensor::getClass()
+inline std::string FactorySensor::getClass() const
 {
   return "FactorySensor";
 }
@@ -240,7 +241,7 @@ typedef Factory<SensorBase,
                 const ParamsServer&> AutoConfFactorySensor;
 
 template<>
-inline std::string AutoConfFactorySensor::getClass()
+inline std::string AutoConfFactorySensor::getClass() const
 {
   return "AutoConfFactorySensor";
 }

include/core/solver/factory_solver.h

@@ -214,7 +214,7 @@ typedef Factory<SolverManager,
                 const ParamsServer&> FactorySolver;
 
 template<>
-inline std::string FactorySolver::getClass()
+inline std::string FactorySolver::getClass() const
 {
   return "FactorySolver";
 }

include/core/tree_manager/factory_tree_manager.h

@@ -23,7 +23,7 @@ struct ParamsTreeManagerBase;
 typedef Factory<ParamsTreeManagerBase,
         const std::string&> FactoryParamsTreeManager;
 template<>
-inline std::string FactoryParamsTreeManager::getClass()
+inline std::string FactoryParamsTreeManager::getClass() const
 {
     return "FactoryParamsTreeManager";
 }
@@ -33,7 +33,7 @@ typedef Factory<TreeManagerBase,
         const std::string&,
         const ParamsTreeManagerBasePtr> FactoryTreeManager;
 template<>
-inline std::string FactoryTreeManager::getClass()
+inline std::string FactoryTreeManager::getClass() const
 {
   return "FactoryTreeManager";
 }
@@ -46,7 +46,7 @@ typedef Factory<TreeManagerBase,
         const std::string&,
         const ParamsServer&> AutoConfFactoryTreeManager;
 template<>
-inline std::string AutoConfFactoryTreeManager::getClass()
+inline std::string AutoConfFactoryTreeManager::getClass() const
 {
     return "AutoConfFactoryTreeManager";
 }

[Joan Vallvé Navarro]

This had to be also applied to the ROS factories.

[Joan Solà Ortega]

@joanvallve you take care of it?

[Joan Vallvé Navarro]

Yes, done!