# HG changeset patch # User nemo # Date 1302535330 14400 # Node ID 932307228d05551c9deb31468fa395471c7efecf # Parent 090a8b8d1083d8328ec3f5639d384b2900eab46d Change a few iterations over list to what is apparently a modification safe syntax, may fix issue #208 and #217 - needs testing of course. diff -r 090a8b8d1083 -r 932307228d05 QTfrontend/chatwidget.cpp --- a/QTfrontend/chatwidget.cpp Mon Apr 11 01:30:43 2011 +0200 +++ b/QTfrontend/chatwidget.cpp Mon Apr 11 11:22:10 2011 -0400 @@ -409,10 +409,8 @@ void HWChatWidget::nickRemoved(const QString& nick) { QList items = chatNicks->findItems(nick, Qt::MatchExactly); - for(QList::iterator it=items.begin(); it!=items.end();) { - chatNicks->takeItem(chatNicks->row(*it)); - ++it; - } + QListIterator it(items); + while(it.hasNext()) chatNicks->takeItem(chatNicks->row(it.next())); lblCount->setText(QString::number(chatNicks->count())); } diff -r 090a8b8d1083 -r 932307228d05 QTfrontend/game.cpp --- a/QTfrontend/game.cpp Mon Apr 11 01:30:43 2011 +0200 +++ b/QTfrontend/game.cpp Mon Apr 11 11:22:10 2011 -0400 @@ -82,8 +82,8 @@ if (m_pTeamSelWidget) { - QList teams = m_pTeamSelWidget->getPlayingTeams(); - for(QList::iterator it = teams.begin(); it != teams.end(); ++it) + QListIterator it(m_pTeamSelWidget->getPlayingTeams()); + while(it.hasNext()) { HWProto::addStringToBuffer(buf, QString("eammloadt %1").arg(ammostr.mid(0, cAmmoNumber))); HWProto::addStringToBuffer(buf, QString("eammprob %1").arg(ammostr.mid(cAmmoNumber, cAmmoNumber))); @@ -91,7 +91,8 @@ HWProto::addStringToBuffer(buf, QString("eammreinf %1").arg(ammostr.mid(3 * cAmmoNumber, cAmmoNumber))); if(!gamecfg->schemeData(21).toBool()) HWProto::addStringToBuffer(buf, QString("eammstore")); HWProto::addStringListToBuffer(buf, - (*it).TeamGameConfig(gamecfg->getInitHealth())); + it.next().TeamGameConfig(gamecfg->getInitHealth())); + ; } } RawSendIPC(buf); @@ -389,9 +390,9 @@ if (m_pTeamSelWidget) { QByteArray buf; - QList teams = m_pTeamSelWidget->getPlayingTeams(); - for(QList::iterator it = teams.begin(); it != teams.end(); ++it) - HWProto::addStringToBuffer(buf, QString("eteamgone %1").arg((*it).TeamName)); + QListIterator it(m_pTeamSelWidget->getPlayingTeams()); + while(it.hasNext()) + HWProto::addStringToBuffer(buf, QString("eteamgone %1").arg(it.next().TeamName)); RawSendIPC(buf); } } diff -r 090a8b8d1083 -r 932307228d05 QTfrontend/hwform.cpp --- a/QTfrontend/hwform.cpp Mon Apr 11 01:30:43 2011 +0200 +++ b/QTfrontend/hwform.cpp Mon Apr 11 11:22:10 2011 -0400 @@ -593,11 +593,10 @@ curTeamSelWidget = ui.pageNetGame->pNetTeamsWidget; } - QList teams = curTeamSelWidget->getDontPlayingTeams(); QStringList tmnames; - for(QList::iterator it = teams.begin(); it != teams.end(); ++it) { - tmnames += it->TeamName; - } + QListIterator it(curTeamSelWidget->getDontPlayingTeams()); + while(it.hasNext()) tmnames += it.next().TeamName; + //UpdateTeamsLists(&tmnames); // FIXME: still need more work if teamname is updated while configuring UpdateTeamsLists(); diff -r 090a8b8d1083 -r 932307228d05 QTfrontend/teamselect.cpp --- a/QTfrontend/teamselect.cpp Mon Apr 11 01:30:43 2011 +0200 +++ b/QTfrontend/teamselect.cpp Mon Apr 11 11:22:10 2011 -0400 @@ -243,7 +243,6 @@ void TeamSelWidget::resetPlayingTeams(const QList& teamslist) { - QList::iterator it; //for(it=curPlayingTeams.begin(); it!=curPlayingTeams.end(); it++) { //framePlaying->removeTeam(*it); //} @@ -256,9 +255,8 @@ frameDontPlaying->resetTeams(); curDontPlayingTeams.clear(); - for (QList::ConstIterator it = teamslist.begin(); it != teamslist.end(); ++it ) { - addTeam(*it); - } + QListIterator it(teamslist); + while(it.hasNext()) addTeam(it.next()); } bool TeamSelWidget::isPlaying(HWTeam team) const