Complete.Org: Mailing Lists: Archives: freeciv-dev: May 2000:
[Freeciv-Dev] Re: [FreeCiv-Cvs] thue: Remove the restriction that the n
Home

[Freeciv-Dev] Re: [FreeCiv-Cvs] thue: Remove the restriction that the n

[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: [FreeCiv-Cvs] thue: Remove the restriction that the number of star... (PR#372)
From: Jeff Mallatt <jjm@xxxxxxxxxxxx>
Date: Thu, 18 May 2000 09:30:43 -0400

At 2000/05/17 18:37 , Thue Janus Kristensen wrote:
>On Wed, 17 May 2000, Jeff Mallatt wrote:
>> At 2000/05/17 09:41 , freeciv@xxxxxxxxxxxxxxxxxxx wrote:
>> >This is an automated notification of a change to freeciv cvs, 
>> >on Wed May 17 06:32:07 PDT 2000 = Wed May 17 13:32:07 2000 (GMT)
>> >by Thue Janus Kristensen <thue@xxxxxxx>
>> >
>> >---- Files affected:
>> >
>> >freeciv/server civserver.c gamehand.c maphand.c stdinhand.c
>> >
>> >---- Log message:
>> >
>> >Remove the restriction that the number of starting positions in a scenario
>> >must be >= MAX_NUM_PLAYERS.
>> >This makes the scenarios work wotk CVS again after MAX_NUM_PLAYERS was
>> >increased from 14 to 30.
>> 
>> Well, actually this does not make scenarios work.  Try loading a scenario,
>> and setting aifill to 20.
>> 
>> jjm
>
>Yes it does. It just reduces aifill to the number of starting positions,
>which I think is ok (at least intented). Works for me at least.

The prior code handled the case by generating random starting locations,
which to me is the right solution.

jjm




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