Complete.Org: Mailing Lists: Archives: freeciv-dev: June 2005:
[Freeciv-Dev] (PR#13360) Choose random tech and future tech
Home

[Freeciv-Dev] (PR#13360) Choose random tech and future tech

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Subject: [Freeciv-Dev] (PR#13360) Choose random tech and future tech
From: "Mateusz Stefek" <mstefek@xxxxxxxxx>
Date: Tue, 28 Jun 2005 10:39:03 -0700
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=13360 >

> [jdorje - Tue Jun 28 17:31:07 2005]:
> 
> Mateusz Stefek wrote:
> > <URL: http://bugs.freeciv.org/Ticket/Display.html?id=13360 >
> > 
> > choose_random_tech can't set A_FUTURE and this leads to many bugs.
> > For example: http://pubserver.freeciv.org/games/443089
> > 
> > Patch attached.
> 
> I don't understand.  Of course choose_random_tech can set A_FUTURE, if
> it's reachable.  If it's not reachable we don't want to choose it anyway.
> 
> You can see this in action in any late game.  Once all techs are found
> and there's no goal, choose_random_tech chooses a new tech for you each
> turn.  And it always chooses A_FUTURE.
> 
> -jason
> 
choose_random_tech() uses choose_tech().
choose_tech ignores A_FUTURE as it's argument.
Note that my patch modifies choose_tech().

I don't know if get_invention() can be called on A_FUTURE
at least in CVS it returns TECH_UNKNOWN
--
mateusz



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