diff options
author | Andrei Karas <akaras@inbox.ru> | 2011-05-26 19:15:48 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2011-05-26 19:15:48 +0300 |
commit | aeef06d5d58dcc6f9339bf6ef5a87e06a4827b17 (patch) | |
tree | 463861d544a1b6bb4fbab4f5176fe812e6b58d97 | |
parent | ce6905ac29998890ed41dc3039bb548a440e2608 (diff) | |
download | manaplus-aeef06d5d58dcc6f9339bf6ef5a87e06a4827b17.tar.gz manaplus-aeef06d5d58dcc6f9339bf6ef5a87e06a4827b17.tar.bz2 manaplus-aeef06d5d58dcc6f9339bf6ef5a87e06a4827b17.tar.xz manaplus-aeef06d5d58dcc6f9339bf6ef5a87e06a4827b17.zip |
Fix some imageset memory leaks.
-rw-r--r-- | src/gui/gui.cpp | 2 | ||||
-rw-r--r-- | src/particleemitter.cpp | 10 | ||||
-rw-r--r-- | src/particleemitter.h | 3 | ||||
-rw-r--r-- | src/resources/resourcemanager.cpp | 10 |
4 files changed, 24 insertions, 1 deletions
diff --git a/src/gui/gui.cpp b/src/gui/gui.cpp index 9df692515..a23da8157 100644 --- a/src/gui/gui.cpp +++ b/src/gui/gui.cpp @@ -264,6 +264,8 @@ void Gui::setUseCustomCursor(bool customCursor) SDL_ShowCursor(SDL_DISABLE); // Load the mouse cursor + if (mMouseCursors) + mMouseCursors->decRef(); mMouseCursors = Theme::getImageSetFromTheme("mouse.png", 40, 40); if (!mMouseCursors) diff --git a/src/particleemitter.cpp b/src/particleemitter.cpp index 33c2f1d92..5f107abb7 100644 --- a/src/particleemitter.cpp +++ b/src/particleemitter.cpp @@ -210,6 +210,7 @@ ParticleEmitter::ParticleEmitter(xmlNodePtr emitterNode, Particle *target, logger->log1("Error: no valid imageset"); continue; } + mTempSets.push_back(imageset); // Get animation frames for_each_xml_child_node(frameNode, propertyNode) @@ -293,6 +294,7 @@ ParticleEmitter::ParticleEmitter(xmlNodePtr emitterNode, Particle *target, logger->log1("Error: no valid imageset"); continue; } + mTempSets.push_back(imageset); // Get animation frames for_each_xml_child_node(frameNode, propertyNode) @@ -437,6 +439,14 @@ ParticleEmitter & ParticleEmitter::operator=(const ParticleEmitter &o) ParticleEmitter::~ParticleEmitter() { + for (std::list<ImageSet*>::iterator + i = mTempSets.begin(); + i != mTempSets.end(); ++i) + { + if (*i) + (*i)->decRef(); + } + if (mParticleImage) mParticleImage->decRef(); } diff --git a/src/particleemitter.h b/src/particleemitter.h index ed6518f0e..5a68584c6 100644 --- a/src/particleemitter.h +++ b/src/particleemitter.h @@ -32,6 +32,7 @@ #include <list> class Image; +class ImageSet; class Map; class Particle; @@ -150,5 +151,7 @@ class ParticleEmitter /** List of emitters the spawned particles are equipped with */ std::list<ParticleEmitter> mParticleChildEmitters; + + std::list<ImageSet*> mTempSets; }; #endif diff --git a/src/resources/resourcemanager.cpp b/src/resources/resourcemanager.cpp index 79231a744..5552b64b1 100644 --- a/src/resources/resourcemanager.cpp +++ b/src/resources/resourcemanager.cpp @@ -138,8 +138,16 @@ ResourceManager::~ResourceManager() } #endif if (iter->second) + { cleanUp(iter->second); - ++iter; + ResourceIterator toErase = iter; + ++iter; + mResources.erase(toErase); + } + else + { + ++iter; + } } } |