Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2001:
[Freeciv-Dev] Re: PATCH: AI cleanup Version 2

[Freeciv-Dev] Re: PATCH: AI cleanup Version 2

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxxxxxxx
Cc: Petr Baudis <pasky@xxxxxxxxxxx>, Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>, freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: PATCH: AI cleanup Version 2
From: Raahul Kumar <raahul_da_man@xxxxxxxxx>
Date: Sat, 24 Nov 2001 06:00:41 -0800 (PST)

--- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, Nov 23, 2001 at 09:08:22PM -0800, Raahul Kumar wrote:
> > 
> > --- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > 
> > Do you have any problems with the current patch I sent your way? If so,
> > I would appreciate comments.
> To be honest I'm a bit lost. Suddenly three people work on the ai
> movement code and produce patches like there is no tomorrow (which is
> good). How does all the patches relate to each other?
> The changes in aicleanupv2.diff are ok.

Thanks. Can you apply them soon? My code stands on its own as a unit.
You can apply it without worries as to the effect on the other movement
code people. In fact, Petr's assumes my patch has been applied.

I'd like to hold off sending the 3rd patch in the series until you've either
applied v1 & 2 or asked me to amend them.

What is currently happening is a general cleanup of the server ai, focusing
exclusively on the movement code ( including an igter unit fix, but that
on what GB says). 

I also had a go at renemaing the variable names of the find_something_to kill
and other ai funcs from a,b,c,d,e,f etc to more descriptive names. That is
Petr comes in. I asked about better names, and an explanation of what those
did. Petr chipped in with comments I believe are correct(so far I've found no
holes in his logic).

I believe Petr understands those ai functions. 

Do You Yahoo!?
Yahoo! GeoCities - quick and easy web site hosting, just $8.95/month.

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