[Freeciv-Dev] Re: init_techs integrated patch (PR#999)

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Tue, Oct 09, 2001 at 08:22:35AM -0700, Arien Malec wrote:
> 
> --- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > > +    int *init_techs; /* Advances to assign to all players at game
> > > +   start. A_LAST terminated.*/
> > 
> > Why is this dynamic and not init_techs[MAX_NUM_INIT_TECHS + 1]? Or
> > should MAX_NUM_INIT_TECHS be named MAX_NUM_NATION_INIT_TECHS?
> 
> Probably the latter. 8 (the value for MAX_NUM_INIT_TECHS) is too low for
> global_init_techs. E.g., if you wanted to test modern warfare, you would need:
> 
> writing & university (library & univ)
> currency, banking, econ (marketplace, etc.)
> conscription, metallurgy, tactics (rifle, etc.)
> railroad, bridge building
> etc. etc. etc.
> 

> I could statically allocate to MAX_NUM_TECH_LIST?

It is hard to judge memory wasting versus easy memory
handling. MAX_NUM_TECH_LIST is currently 10. 10 is ok. Do you plan to
increase MAX_NUM_TECH_LIST? If yes, how large?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "On the eigth day, God started debugging"