RemoteGui: Use much simpler extension interface (overload instead of template specialization)

source/RobotAPI/libraries/ControllerUIUtility/CartesianWaypointControllerConfig/RemoteGui.cpp

@@ -155,12 +155,9 @@ namespace armarx::RemoteGui
                    .addChild(new RemoteGui::HSpacer)
                );
     }
-}
-namespace armarx::RemoteGui::detail::_getValue
-{
-    void GetValueOutputParameter <CartesianWaypointControllerConfig, void>::call(
-        CartesianWaypointControllerConfig& cfg,
-        RemoteGui::ValueMap const& values, std::string const& name)
+
+    void getValue(CartesianWaypointControllerConfig& cfg,
+                  RemoteGui::ValueMap const& values, std::string const& name)
     {
         cfg.maxPositionAcceleration     = getValue<float>(values, name + "_maxAcc_Pos");
         cfg.maxOrientationAcceleration  = getValue<float>(values, name + "_maxAcc_Ori");

source/RobotAPI/libraries/ControllerUIUtility/CartesianWaypointControllerConfig/RemoteGui.h

@@ -31,15 +31,7 @@ namespace armarx::RemoteGui
     detail::GroupBoxBuilder makeConfigGui(
         const std::string& name,
         const CartesianWaypointControllerConfig& val);
-}
 
-namespace armarx::RemoteGui::detail::_getValue
-{
-    template<>
-    struct GetValueOutputParameter <CartesianWaypointControllerConfig, void> : std::true_type
-    {
-        static void call(
-            CartesianWaypointControllerConfig& cfg,
-            RemoteGui::ValueMap const& values, std::string const& name);
-    };
+    void getValue(CartesianWaypointControllerConfig& cfg,
+                  RemoteGui::ValueMap const& values, std::string const& name);
 }

source/RobotAPI/libraries/ControllerUIUtility/NJointCartesianWaypointControllerConfig/RemoteGui.cpp

@@ -49,12 +49,9 @@ namespace armarx::RemoteGui
 
         return builder;
     }
-}
-namespace armarx::RemoteGui::detail::_getValue
-{
-    void GetValueOutputParameter <NJointCartesianWaypointControllerRuntimeConfig, void>::call(
-        NJointCartesianWaypointControllerRuntimeConfig& cfg,
-        RemoteGui::ValueMap const& values, std::string const& name)
+
+    void getValue(NJointCartesianWaypointControllerRuntimeConfig& cfg,
+                  RemoteGui::ValueMap const& values, std::string const& name)
     {
         getValue(cfg.wpCfg,          values, name);
         getValue(cfg.forceThreshold, values, name + "_forceThreshold");

source/RobotAPI/libraries/ControllerUIUtility/NJointCartesianWaypointControllerConfig/RemoteGui.h

@@ -36,15 +36,7 @@ namespace armarx::RemoteGui
     NJointCartesianWaypointControllerRuntimeConfig getValue<NJointCartesianWaypointControllerRuntimeConfig>(
         ValueMap const& values,
         std::string const& name);
-}
 
-namespace armarx::RemoteGui::detail::_getValue
-{
-    template<>
-    struct GetValueOutputParameter <NJointCartesianWaypointControllerRuntimeConfig, void> : std::true_type
-    {
-        static void call(
-            NJointCartesianWaypointControllerRuntimeConfig& cfg,
-            RemoteGui::ValueMap const& values, std::string const& name);
-    };
+    void getValue(NJointCartesianWaypointControllerRuntimeConfig& cfg,
+                  RemoteGui::ValueMap const& values, std::string const& name);
 }