Merge lp:~hjd/widelands/simplify-loop into lp:widelands

[Hans Joachim Desserud]

Basically implementing the solution suggested in the bug report, plus trim out a couple more intermediary variables.

[SirVer (sirver)]

review: Approve

=== modified file 'src/game_io/game_player_economies_data_packet.cc'
--- src/game_io/game_player_economies_data_packet.cc	2014-07-28 14:17:07 +0000
+++ src/game_io/game_player_economies_data_packet.cc	2014-08-02 11:17:59 +0000
@@ -50,13 +50,10 @@
 			iterate_players_existing(p, nr_players, game, player)
 				try {
 					Player::Economies & economies = player->m_economies;
-					uint16_t const nr_economies = economies.size();
-					Player::Economies ecos(nr_economies);
-					for (Economy * temp_eco : ecos) {
+					for (uint32_t i = 0; i < economies.size(); ++i) {
 						uint32_t value = fr.Unsigned32();
 						if (value < 0xffffffff) {
 							if (upcast(Flag const, flag, map[value].get_immovable())) {
-								temp_eco = flag->get_economy();
 								EconomyDataPacket d(flag->get_economy());
 								d.Read(fr);
 							} else {