Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: Notes and serial numbers
Home

[Freeciv-Dev] Re: Notes and serial numbers

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Per I. Mathisen" <Per.Inge.Mathisen@xxxxxxxxxxx>
Cc: freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: Notes and serial numbers
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 3 Sep 2001 14:42:36 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Mon, Sep 03, 2001 at 01:45:08PM +0200, Per I. Mathisen wrote:
> On Sun, 2 Sep 2001, Raimar Falke wrote:
> > Problem: the server currently doesn't respond to each packet it gets
> > from the client. There is no packet sent back for example if a unit
> > tried to do some stupid (into the ocean and no ship) move. Any client
> > side automated code however has to know if the request succeeded or
> > not. One way is to add an "error" packet to every error path. Another
> > way is to sent a generic "execution finished" packet after every set
> > (can be empty) of packet which were sent during the normal execution
> > of the command. This would allow the client to wait for this
> > "execution finished" packet and than check the new position of the
> > unit and sees if it had been changed or not. To have more than one
> > packet on the fly between client and server serial numbers are needed
> > to match "execution finished" packets with the corresponding
> > commands.
> 
> Good suggestion, but since human opponents do not need this, can it be
> optional? Client AIs tell the server that it needs serials, and the server
> adds serials to all packets going to that player. It adds more overhead,
> but gives smaller bandwidth consumption.
> 
> Perhaps that part of the code could even be put into a macro, and that
> macro be disable-able through configure, for those who want speed rather
> than bandwidth conservation?

Well the patch doesn't exists yet. If I do the patch as I original
thought this is possible by doing a 'FREECIV_CAPS="+1.11.6 conn_info
pop_cost turn attributes" ./civ -t trident'. Another way is to change:
void init_our_capability(void)
{
  char *s;

  s = getenv("FREECIV_CAPS");
  if (s == NULL) {
    s = CAPABILITY;
  }
  sz_strlcpy(our_capability_internal, s);
+ if(want_sno)
+    sz_strlcat(our_capability_internal, " sno");
}

So I see no problem of disabling this.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "I heard if you play the NT-4.0-CD backwards, you get a satanic message."
 "That's nothing, if you play it forward, it installs NT-4.0"


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