0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

don't use a constant end iterator to terminate an erasing loop

Browse source
This commit is contained in:
Patrick Parker 2007-10-17 01:41:01 +00:00
parent e6d86722f1
commit 96def93067
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/unit.cpp
View file

@ -771,8 +771,8 @@ void unit::remove_ability_by_id(const std::string &ability)
{
config* abil = cfg_.child("abilities");
if(abil) {
config::all_children_iterator i = abil->ordered_begin(), i_end = abil->ordered_end();
while(i != i_end) {
config::all_children_iterator i = abil->ordered_begin();
while(i != abil->ordered_end()) {
if(i.get_child()["id"] == ability) {
i = abil->erase(i);
} else {