widelands-dev team mailing list archive

[TiborB <tiborb95@xxxxxxxxx>]

Indeed. I cannot work with bzr now, but I would suggest slightly alternative design. Like this:

	for (uint8_t j = 1; j <= plr_in_game; ++j) {
		if (!game().get_player(j)) {
			continue;
		}
		TeamNumber const tm = game().get_player(j)->team_number();


	// now we test all players to identify 'attackable' ones
	for (uint8_t j = 1; j <= plr_in_game; ++j) {
		// if it's me or player's slot unoccupied
		if (pn == j || !game().get_player(j)) {
			player_attackable[j - 1] = false;

If you want, you can change your branch now and I will approve it, or I will change it myself tonight or tomorrow :)
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_null_enemy/+merge/275673
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_null_enemy.