Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2001:
[Freeciv-Dev] Re: [Patch] Introduction of turns
Home

[Freeciv-Dev] Re: [Patch] Introduction of turns

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Reinier Post <rp@xxxxxxxxxx>
Cc: Freeciv developers <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: [Patch] Introduction of turns
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 27 Aug 2001 09:45:35 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Mon, Aug 27, 2001 at 09:33:58AM +0200, Reinier Post wrote:
> On Mon, Aug 27, 2001 at 08:57:02AM +0200, Raimar Falke wrote:
> > On Sat, Aug 11, 2001 at 12:26:35PM +0200, Raimar Falke wrote:
> > > 
> > > For client side ai and maybe also for other things the definition of
> > > the current turn is needed. The attached patch adds this. It is
> > > straightforward except this piece:
> > 
> > Any more comments on this? Questions?
> 
> I haven't looked at the code, sorry ...
> 
> You still haven't provided an argument why the client needs to be able
> to know the current year, but it seems reasonable.

The year is insufficient. A simpler form is required.

> Generally I feel that state should be kept in the server and the client
> should only cache, 

Ack. The patch wouldn't change anything on this.

> and that agents should keep the game state in the server

The attribute patch will allow this.

, but that discussion is complete, and I do not have any code to
> provide an alternative.  So if your code works and provides new
> funvtionality, please put it in.

The display of the current turn in the gtk client was only a debugging
thing. Should I leave it in?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  What's nice about GUI is that you see what you manipulate.
  What's bad about GUI is that you can only manipulate what you see.


[Prev in Thread] Current Thread [Next in Thread]