[Freeciv-Dev] Re: Tech cost patch v16 (PR#1082)

[Juha Litola <slave@xxxxxxxxxxxx>]

On Sun, Dec 16, 2001 at 04:57:05PM +0100, Raimar Falke wrote:
> 
> Can't precalc_tech_data use num_requirements?
Yes it can, nowadays.

> find_requirements, num_requirements should get a "tech_" as prefix.
> Type of "tech" in precalc_tech_data should be Tech_Type_id.
ok.

> I agree that a
> 
>   int is_tech_a_requirement_tech_for(struct player *pplayer, Tech_Type_id 
> goal, Tech_Type_id to_test);
> 
> would be nice. This allows to make find_requirements a static
> function. is_tech_a_requirement_tech_for could be implemented brute
> force or with a cache which is contained in the player struct.
I don't currently have time to make patch as I'll go on vacation for 
couple of weeks and have some work to catch. I can make a version in 
some time January, but I think current version could be added to cvs 
before that. It is after all functional, it can just be optimized
to be faster.

I can make patch to implement first to observations, but I think 
you'll do it equally fast if you are going to integrate patch.

-- 
// Juha Litola

pgpC3NQt2ipsK.pgp
Description: PGP signature