replaced use of vector with list, small fixes

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

@@ -2,6 +2,7 @@
 
 #include <optional>
 #include <string>
+#include <list>
 
 #include <IceUtil/UUID.h>
 
@@ -696,7 +697,7 @@ namespace armarx::plugins
     std::optional<skills::FluxioSkill>
     SkillManagerComponentPlugin::getSkill(const std::string& id)
     {
-        auto skillsEntry = fluxioDC.skills.find(id);
+        const auto& skillsEntry = fluxioDC.skills.find(id);
         if (skillsEntry != fluxioDC.skills.end())
         {
             return skillsEntry->second;
@@ -721,7 +722,7 @@ namespace armarx::plugins
         // oldSkill->native = skill.native; Not changeable
 
         // ---update the parameters lists entirelys
-        std::vector<std::string> toRemoveIds;
+        std::list<std::string> toRemoveIds;
         // fill the toRemoveIds vector with all old skills parameters
         for (const auto& parameter : oldSkill.parameters)
         {