Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2002:
[Freeciv-Dev] Re: agent assertion failed (PR#1259)
Home

[Freeciv-Dev] Re: agent assertion failed (PR#1259)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Cc: bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: agent assertion failed (PR#1259)
From: rf13@xxxxxxxxxxxxxxxxxxxxxx
Date: Mon, 11 Feb 2002 07:11:54 -0800 (PST)

On Mon, Feb 11, 2002 at 02:29:34PM +0000, Ben Webb wrote:
> On Mon, 11 Feb 2002, Raimar Falke wrote:
> 
> > Mhhh ... I looks like the test has be to moved:
> > 
> >   if (packet->you_can_join) {
> >     freelog(LOG_VERBOSE, "join game accept:%s", packet->message);
> >     aconnection.established = 1;
> >     game.conn_id = packet->conn_id;
> > +   agents_game_joined();
> >   } else {
> 
>       That won't work for connecting to or from old servers that don't 
> support the turn, attributes, processing_packets or tile_trade 
> capabilities, surely. Won't the asserts in agents_game_joined() then be 
> triggered? Can't you either disable the agents code if these attributes 
> aren't present, or make them mandatory capabilities (in which case the 
> asserts "should" never be triggered) ?

To be honest I don't know what will happen if the agents are used
without turns, processing_packets, attributes and so on.

The asserts are some kind of "mandatory capability" ;)

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "We've all heard that a million monkeys banging on a million typewriters
  will eventually reproduce the entire works of Shakespeare.
  Now, thanks to the Internet, we know this is not true."



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