Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2004:
[Freeciv-Dev] Re: (PR#11106) revolution bugs
Home

[Freeciv-Dev] Re: (PR#11106) revolution bugs

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Subject: [Freeciv-Dev] Re: (PR#11106) revolution bugs
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Sat, 20 Nov 2004 11:07:44 -0800
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=11106 >

And there's a second revolution problem.  When revolution_finishes is 
reset to -1 the information isn't getting properly sent to the client. 
I would have though this would be done automatically at the start of the 
phase, but I guess there's no harm in doing it manually.

This patch fixes both bugs.

jason


Index: server/plrhand.c
===================================================================
RCS file: /home/freeciv/CVS/freeciv/server/plrhand.c,v
retrieving revision 1.338
diff -u -r1.338 plrhand.c
--- server/plrhand.c    20 Nov 2004 17:48:24 -0000      1.338
+++ server/plrhand.c    20 Nov 2004 19:06:37 -0000
@@ -915,7 +915,7 @@
   } else if (game.revolution_length == 0) {
     pplayer->revolution_finishes = game.turn + myrand(5) + 1;
   } else {
-    pplayer->revolution_finishes = game.revolution_length;
+    pplayer->revolution_finishes = game.turn + game.revolution_length;
   }
 
   freelog(LOG_DEBUG,
@@ -1036,6 +1036,7 @@
     freelog(LOG_DEBUG, "Update: resetting revofin for %s.",
            pplayer->name);
     pplayer->revolution_finishes = -1;
+    send_player_info(pplayer, pplayer);
   }
 }
 

[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] Re: (PR#11106) revolution bugs, Jason Short <=