From fd191dd616b85ca27f2ef838fc68b41fdae0eae0 Mon Sep 17 00:00:00 2001 From: smatz Date: Tue, 9 Jun 2009 23:22:37 +0000 Subject: [PATCH] (svn r16546) -Codechange: use array member instead of allocating in costructor and freeing in desctructor --- src/misc/hashtable.hpp | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/misc/hashtable.hpp b/src/misc/hashtable.hpp index 05430f446a..2e524e6322 100644 --- a/src/misc/hashtable.hpp +++ b/src/misc/hashtable.hpp @@ -12,7 +12,7 @@ struct CHashTableSlotT Titem_ *m_pFirst; - CHashTableSlotT() : m_pFirst(NULL) {} + FORCEINLINE CHashTableSlotT() : m_pFirst(NULL) {} /** hash table slot helper - clears the slot by simple forgetting its items */ FORCEINLINE void Clear() {m_pFirst = NULL;} @@ -133,19 +133,16 @@ protected: * Titem contains pointer to the next item - GetHashNext(), SetHashNext() */ typedef CHashTableSlotT Slot; - Slot *m_slots; // here we store our data (array of blobs) - int m_num_items; // item counter + Slot m_slots[Tcapacity]; ///< here we store our data (array of blobs) + int m_num_items; ///< item counter public: /* default constructor */ - FORCEINLINE CHashTableT() - { - /* construct all slots */ - m_slots = new Slot[Tcapacity]; - m_num_items = 0; - } + FORCEINLINE CHashTableT() : + m_num_items(0) + { } - ~CHashTableT() {delete [] m_slots; m_num_items = 0; m_slots = NULL;} + FORCEINLINE ~CHashTableT() { } protected: /** static helper - return hash for the given key modulo number of slots */ @@ -168,7 +165,10 @@ public: FORCEINLINE int Count() const {return m_num_items;} /** simple clear - forget all items - used by CSegmentCostCacheT.Flush() */ - FORCEINLINE void Clear() const {for (int i = 0; i < Tcapacity; i++) m_slots[i].Clear();} + FORCEINLINE void Clear() + { + for (size_t i = 0; i < lengthof(m_slots); i++) m_slots[i].Clear(); + } /** const item search */ const Titem_ *Find(const Tkey& key) const