diff options
author | Jared Adams <jaxad0127@gmail.com> | 2010-05-06 23:11:21 -0600 |
---|---|---|
committer | Jared Adams <jaxad0127@gmail.com> | 2010-05-21 12:27:32 -0600 |
commit | 910ea1476664d8a11d57eaf8ff7f281bcfacd184 (patch) | |
tree | ffed95b63dd38a03b058738f26ad89fb38ed1dfa /src/particlecontainer.cpp | |
parent | d34dda737ed8af5135c49e269f5ad608cea8d511 (diff) | |
download | mana-910ea1476664d8a11d57eaf8ff7f281bcfacd184.tar.gz mana-910ea1476664d8a11d57eaf8ff7f281bcfacd184.tar.bz2 mana-910ea1476664d8a11d57eaf8ff7f281bcfacd184.tar.xz mana-910ea1476664d8a11d57eaf8ff7f281bcfacd184.zip |
Fix some issues found by Cppcheck
Diffstat (limited to 'src/particlecontainer.cpp')
-rw-r--r-- | src/particlecontainer.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/particlecontainer.cpp b/src/particlecontainer.cpp index 175c1e87..6900539d 100644 --- a/src/particlecontainer.cpp +++ b/src/particlecontainer.cpp @@ -71,14 +71,17 @@ void ParticleList::addLocally(Particle *particle) void ParticleList::removeLocally(Particle *particle) { - for (std::list<Particle *>::iterator it = mElements.begin(); - it != mElements.end(); it++) + std::list<Particle *>::iterator it, it_end; + for (it = mElements.begin(), it_end = mElements.end(); + it != it_end;) { if (*it == particle) { (*it)->kill(); - mElements.erase(it); + it = mElements.erase(it); } + else + it++; } } |