[Freeciv-commits] r29629 - in /trunk/server: plrhand.h srv_main.c

[cazfi74]

Author: cazfi
Date: Fri Aug 21 18:30:39 2015
New Revision: 29629

URL: http://svn.gna.org/viewcvs/freeciv?rev=29629&view=rev
Log:
Do not progress, nor finish, revolution for a dead player.

See bug #23803

Modified:
    trunk/server/plrhand.h
    trunk/server/srv_main.c

Modified: trunk/server/plrhand.h
URL: 
http://svn.gna.org/viewcvs/freeciv/trunk/server/plrhand.h?rev=29629&r1=29628&r2=29629&view=diff
==============================================================================
--- trunk/server/plrhand.h      (original)
+++ trunk/server/plrhand.h      Fri Aug 21 18:30:39 2015
@@ -109,7 +109,17 @@
 #define phase_players_iterate_end\
     }\
   } shuffled_players_iterate_end;\
-} while (0)
+} while (FALSE);
+
+#define alive_phase_players_iterate(pplayer) \
+do { \
+  phase_players_iterate(pplayer) { \
+    if (pplayer->is_alive) {
+
+#define alive_phase_players_iterate_end \
+    } \
+  } phase_players_iterate_end \
+} while (FALSE);
 
 bool civil_war_possible(struct player *pplayer, bool conquering_city,
                         bool honour_server_option);

Modified: trunk/server/srv_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/trunk/server/srv_main.c?rev=29629&r1=29628&r2=29629&view=diff
==============================================================================
--- trunk/server/srv_main.c     (original)
+++ trunk/server/srv_main.c     Fri Aug 21 18:30:39 2015
@@ -1028,9 +1028,9 @@
   flush_packets();  /* to curb major city spam */
   conn_list_do_unbuffer(game.est_connections);
 
-  phase_players_iterate(pplayer) {
+  alive_phase_players_iterate(pplayer) {
     update_revolution(pplayer);
-  } phase_players_iterate_end;
+  } alive_phase_players_iterate_end;
 
   if (is_new_phase) {
     /* Try to avoid hiding events under a diplomacy dialog */


_______________________________________________
Freeciv-commits mailing list
Freeciv-commits@gna.org
https://mail.gna.org/listinfo/freeciv-commits