build works (Yippie)

source/RobotAPI/libraries/skills/core/FluxioResult.cpp

-#include <stdexcept>
-
-namespace armarx {
-
-    namespace skills {
-    
-        template<typename T, typename E = std::exception>
-
-        class Result {
-            private: bool success_; T result_; E error_; //maybe even an error list for a better stacktrace
-
-            public:
-                Result(const T& res) : success_(true),  result_(res) {}
-
-                Result(const E& err) : success_(false),  error_(err) {}
-
-            bool isSuccess() const {
-            return success_;
-            }
-
-            T getResult() const {
-                if (!success_) {
-                    throw std::logic_error("Result is not successful");
-                }
-            return result_;
-            }
-
-            E getError() const {
-                if (success_) {
-                    throw std::logic_error("Result does not contain an error");
-                }
-                return error_;
-            }
-        };
-    }
-}
\ No newline at end of file

source/RobotAPI/libraries/skills/core/FluxioResult.h

 #include <stdexcept>
+#include <memory>
 
-namespace armarx {
-    namespace skills {
-        
-        template<typename T, typename E = std::exception>
-        class Result {
+namespace armarx
+{
+    namespace skills
+    {
+
+        template <typename T, typename E = std::exception>
+        class Result
+        {
         private:
-            bool success_;
-            T result_;
-            E error_;
+            const bool success;
+            const std::shared_ptr<const T> result;
+            const std::shared_ptr<const E> error;
 
         public:
-            Result(const T& res) : success_(true), result_(res) {}
+            Result(const T& res) : success(true), result(std::make_shared<const T>(res)), error(nullptr)
+            {
+            }
 
-            Result(const E& err) : success_(false), error_(err) {}
+            Result(const E& err) : success(false), result(nullptr), error(std::make_shared<const E>(err))
+            {
+            }
 
-            bool isSuccess() const {}
+            bool isSuccess() const
+            {
+                return success;
+            }
 
-            T getResult() const {}
+            T getResult() const
+            {
+                if (!success)
+                {
+                    throw std::logic_error("Result is not successful");
+                }
+                return *result;
+            }
 
-            E getError() const {}
+            E getError() const
+            {
+                if (success)
+                {
+                    throw std::logic_error("Result does not contain an error");
+                }
+                return *error;
+            }
         };
     }
 }

source/RobotAPI/libraries/skills/manager/SkillManagerComponentPluginUser.cpp

@@ -223,14 +223,14 @@ namespace armarx
     IceUtil::Optional<skills::manager::dto::FluxioSkill>
     SkillManagerComponentPluginUser::getSkill(const std::string& id, const Ice::Current& current)
     {
-        auto skill = this->plugin->getSkill(id);
+        auto result = this->plugin->getSkill(id);
 
-        if (skill.has_value())
+        if (result.isSuccess())
         {
-            const auto& s = skill->toManagerIce();
-            if (s.has_value())
+            const auto& skill = result.getResult().toManagerIce();
+            if (skill.has_value())
             {
-                return *s;
+                return *skill;
             }
         }
         return {};