Implement QueryResult_AutoPtr type which is ACE's reference counted auto_ptr(ACE_Refcounted_Auto_Ptr) for QueryResult pointers.

Use this auto_ptr for every DB queries(except QueryNamedResult yet).

This patch guarantees NO memory leaks from QueryResult pointers.

Thanks to raczman for the idea and for the helping to make this patch.

--HG--
branch : trunk

1 files changed, 1 insertions, 4 deletions

diff --git a/src/game/MapManager.cpp b/src/game/MapManager.cpp
index b57f9b8d2c1..efc59aadbd1 100644
--- a/src/game/MapManager.cpp
+++ b/src/game/MapManager.cpp
@@ -358,12 +358,9 @@ void MapManager::InitMaxInstanceId()
 {
     i_MaxInstanceId = 0;
 
-    QueryResult *result = CharacterDatabase.Query("SELECT MAX(id) FROM instance");
+    QueryResult_AutoPtr result = CharacterDatabase.Query("SELECT MAX(id) FROM instance");
     if (result)
-    {
         i_MaxInstanceId = result->Fetch()[0].GetUInt32();
-        delete result;
-    }
 }
 
 uint32 MapManager::GetNumInstances()