From d9cb0702158fd045285f2c0a904cb31a45a3864a Mon Sep 17 00:00:00 2001 From: Anubisss Date: Sat, 23 Jan 2010 14:45:58 +0100 Subject: 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 --- src/game/MapManager.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'src/game/MapManager.cpp') 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() -- cgit v1.2.3