Complete.Org: Mailing Lists: Archives: freeciv-dev: April 2002:
[Freeciv-Dev] Re: New AI cleanups (PR#1360)
Home

[Freeciv-Dev] Re: New AI cleanups (PR#1360)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxxxxxxx, freeciv-dev@xxxxxxxxxxx
Cc: bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: New AI cleanups (PR#1360)
From: Raahul Kumar <raahul_da_man@xxxxxxxxx>
Date: Wed, 3 Apr 2002 06:44:53 -0800 (PST)

--- rf13@xxxxxxxxxxxxxxxxxxxxxx wrote:
> On Sat, Mar 23, 2002 at 03:57:33AM -0800, Raahul Kumar wrote:
> > The following diffs are included as a replacement for Aicleanup Version 3.
> > No save games are different, and this was done against March 20 CVS. The
> > changes only touch aiunit.c which has not been effected by any changes to
> the
> > AI according to the changelog.
> > 
> > The patches are:
> > 
> > Unit move turns 2
> > Fix Sea Moves
> > 
> > Unit move turns was a result of a discussion between Raimar and myself.
> > Greg(GB) improved the comments, and Raimar actually coded it. I'm just
> > responsible for the comments and the update to current CVS.
> > 
> > Fix Sea moves is a minor cleanup on my part to use SINGLE_MOVE where it
> should
> > be used.Don't bother reading the comment, read the diff. 
> > 
> > If anyone wishes to know, yes, I will send in IGTER cleanups next.
> 
> <nitpicking>
> 
> > + path1 costs: first tile = 3, second tile = 2
> 
> > + path2 costs: first tile=2, second tile=3

This patch is already in CVS. There are two options

1.) You can fix it
2.) I can send in a patch

Let me know which you prefer.


> Should be the same style (i.e. adding spaces).
> 
> > + In spite of the path costs being the same, these two units will arrive
> 
> This is about paths and not units.

I was using them as an illustration. Imagine two units taking these paths. The
point was that one would arrive before the other. I can include those words in
the next patch if you like.

__________________________________________________
Do You Yahoo!?
Yahoo! Tax Center - online filing with TurboTax
http://taxes.yahoo.com/


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