Core/ScriptMgr: use vector instead of list to contain Spell and Aura scripts and hooks

(cherry picked from commit cbb5c073ceb951fa138bb05b55de8b21ffd83063)

1 files changed, 17 insertions, 15 deletions

diff --git a/src/common/Utilities/Util.h b/src/common/Utilities/Util.h
index 6977de5a0c5..d996caf32a5 100644
--- a/src/common/Utilities/Util.h
+++ b/src/common/Utilities/Util.h
@@ -327,33 +327,35 @@ TC_COMMON_API bool StringToBool(std::string const& str);
 
 // simple class for not-modifyable list
 template <typename T>
-class HookList
+class HookList final
 {
-    typedef typename std::list<T>::iterator ListIterator;
     private:
-        typename std::list<T> m_list;
+        typedef std::vector<T> ContainerType;
+
+        ContainerType _container;
+
     public:
-        HookList<T> & operator+=(T t)
-        {
-            m_list.push_back(t);
-            return *this;
-        }
-        HookList<T> & operator-=(T t)
+        typedef typename ContainerType::iterator iterator;
+
+        HookList<T>& operator+=(T t)
         {
-            m_list.remove(t);
+            _container.push_back(t);
             return *this;
         }
+
         size_t size()
         {
-            return m_list.size();
+            return _container.size();
         }
-        ListIterator begin()
+
+        iterator begin()
         {
-            return m_list.begin();
+            return _container.begin();
         }
-        ListIterator end()
+
+        iterator end()
         {
-            return m_list.end();
+            return _container.end();
         }
 };