Complete.Org: Mailing Lists: Archives: freeciv-ai: May 2004:
[freeciv-ai] (PR#8543) AI cannot load transports

[freeciv-ai] (PR#8543) AI cannot load transports

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: per@xxxxxxxxxxx
Subject: [freeciv-ai] (PR#8543) AI cannot load transports
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Thu, 6 May 2004 13:41:19 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: >

> [per - Sun May 02 15:30:10 2004]:
> > [jdorje - Mon Apr 26 04:02:07 2004]:
> > The AI will likely need more changes to work well with the new loading
> > interface.
> What changes?

Well, if we knew what the changes were they'd be easy to make.

However, I suggest the following:

- A consistent AI load/unload method is needed.  Currently one place
uses handle_unit_load while another uses load_unit_onto_transporter. 
The difference is that the latter has no sanity checks.  I'd suggest
adding sanity check assertions to load_unit_onto_transporter.  That way
at least you'll get a useful error message.  But the callers must still
be careful to verify that the load is legal.

- I don't know if any other code still tries to load units by sentrying
them, or unload them by idling them.  I'd suggest grepping the AI code
for SENTRY and IDLE.


[Prev in Thread] Current Thread [Next in Thread]
  • [freeciv-ai] (PR#8543) AI cannot load transports, Jason Short <=