[freeciv-ai] Re: About ai_choose_attacker
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
On Tue, 6 May 2003, Olivier DAVY wrote:
> Great !
> And what about the FIXME for the amortize function (time_to_build) in
> advmilitary.c ?
> Gregory said :
> - the FIXME in the comment of ai_choose_attacker is wrong and should _not_
> be followed. The proper explanation of the function's role can be found
> in the comment for kill_something_with. Instead you can remove the
> fixme and maybe change the function name to something more suitable.
> I am not sure about FIXME in choose_bodyguard, ask Per.
> What should I do for this part ?
Since Per doesn't want to answer, I will assume the responsibility.
*Assuming austere look*
Stuff this comment too. If we are sticking with "units-find-jobs" model,
then I like the current agressive unit selection a lot:
1. choose the best aggressive unit
2. find the job for it
3. choose the best unit for this job (now taking build-costs into
Anyway, there are many other interesting and challenging tasks. For an
example, see Per's email
It needs to be approached with some care though!