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

[Freeciv-Dev] Re: New AI cleanups

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Raahul Kumar <raahul_da_man@xxxxxxxxx>
Cc: Mike Kaufman <kaufman@xxxxxxxxxxxxxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: New AI cleanups
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 3 Apr 2002 16:36:08 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

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

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.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Like the ad says, at 300 dpi you can tell she's wearing a
  swimsuit. At 600 dpi you can tell it's wet. At 1200 dpi you
  can tell it's painted on. I suppose at 2400 dpi you can tell
  if the paint is giving her a rash."
    -- Joshua R. Poulson


[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] Re: New AI cleanups, Raimar Falke <=