Clean up

source/RobotAPI/gui-plugins/SkillManagerPlugin/SkillManagerMonitorWidgetController.h

@@ -80,8 +80,7 @@ namespace armarx
         {
         }
 
-        // When using this constructor, the new item will be _appended_ to the bottom of the tree!
-        // Use with care.
+        // When using this constructor, the new item will be appended to the bottom of the tree!
         SkillExecutionInfoTreeWidgetItem(const skills::SkillExecutionID& id, QTreeWidget* parent) :
             QTreeWidgetItem(parent), executionId(id)
         {

source/RobotAPI/interface/skills/SkillProviderInterface.ice

@@ -136,7 +136,6 @@ module armarx
                 {
                     SkillID skillId;
                     string executorName;
-                    string executionSignature;
                     armarx::core::time::dto::DateTime executionStartedTime;
                     string uuid;
                 };

source/RobotAPI/libraries/armem_skills/server/segment/SkillEventSegment.cpp

@@ -63,7 +63,7 @@ namespace armarx::skills::segment
             [&](const armem::wm::EntityInstance& i)
             {
                 auto event = i.dataAs<armarx::skills::arondto::SkillStatusUpdate>();
-                skills::SkillStatusUpdate up = {};
+                skills::SkillStatusUpdate up;
                 armem::fromAron(event, up);
 
                 if (auto it = ret.find(up.executionId); it != ret.end() && up < it->second)

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

@@ -4,12 +4,6 @@ namespace armarx
 {
     namespace skills
     {
-        /*SkillExecutionID::SkillExecutionID(const SkillID& id,
-                                           const std::string& executorName,
-                                           const armarx::core::time::DateTime& time) :
-            skillId(id), executorName(executorName), executionStartedTime(time)
-        {
-        }*/
 
         skills::manager::dto::SkillExecutionID
         SkillExecutionID::toManagerIce() const

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

@@ -27,12 +27,6 @@ namespace armarx
             static const constexpr char* EXIT_SEPARATOR = "]";
             static const constexpr char* SEPARATOR = "@";
 
-            //SkillExecutionID() = delete;
-
-            /*SkillExecutionID(const SkillID& id,
-                             const std::string& executorName,
-                             const armarx::core::time::DateTime& time);*/
-
             bool
             operator==(const SkillExecutionID& other) const
             {