Main: ControllerManager - use std::vector for ControllerList

paroj · Nov 11, 2023 · f7cd9dd · f7cd9dd
1 parent 356ee5e
commit f7cd9dd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/OgreMain/include/OgreControllerManager.h b/OgreMain/include/OgreControllerManager.h
@@ -56,7 +56,7 @@ namespace Ogre {
     class _OgreExport ControllerManager : public Singleton<ControllerManager>, public ControllerAlloc
     {
     private:
-        typedef std::set<ControllerFloat*> ControllerList;
+        typedef std::vector<ControllerFloat*> ControllerList;
         ControllerList mControllers;
 
         /// Global predefined controller

diff --git a/OgreMain/src/OgreControllerManager.cpp b/OgreMain/src/OgreControllerManager.cpp
@@ -61,7 +61,7 @@ namespace Ogre {
     {
         ControllerFloat* c = OGRE_NEW ControllerFloat(src, dest, func);
 
-        mControllers.insert(c);
+        mControllers.push_back(c);
         return c;
     }
     //-----------------------------------------------------------------------
@@ -206,10 +206,11 @@ namespace Ogre {
     //-----------------------------------------------------------------------
     void ControllerManager::destroyController(ControllerFloat* controller)
     {
-        ControllerList::iterator i = mControllers.find(controller);
+        ControllerList::iterator i = std::find(mControllers.begin(), mControllers.end(), controller);
         if (i != mControllers.end())
         {
-            mControllers.erase(i);
+            std::swap(*i, mControllers.back());
+            mControllers.pop_back();
             OGRE_DELETE controller;
         }
     }