Complete.Org: Mailing Lists: Archives: freeciv-dev: January 2002:
[Freeciv-Dev] [PATCH] aiunit.c ai_manage_explorer cleanup (PR#1210)
Home

[Freeciv-Dev] [PATCH] aiunit.c ai_manage_explorer cleanup (PR#1210)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Cc: bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] [PATCH] aiunit.c ai_manage_explorer cleanup (PR#1210)
From: Petr Baudis <pasky@xxxxxxxxxxx>
Date: Sun, 6 Jan 2002 13:59:40 -0800 (PST)

Hello,

  attached patch cleans the ai_manage_explorer function in aiunit.c of AI.
Behaviour is not changed (autogames SHOULD remain same - I will do tests
tommorow hopefully - now I've some weird problems with machine and I fear to
run autogame now), it was only heavily commented and variable names were
expanded to something meaningful. Oh, yes, and it seems to me that it compiles.

  I think it's ready to actually go in, altough I'm open to objections and
ideas how to extend/correct it even more.

  Note that this patch is obviously only third in very long line. It should be
relatively light stuff and its path to CVS should be IMHO short. However, I'm
not sure about some code (especially not why something is here, but why
something is NOT here ;-), and I would like to receive some comments about it
(the problem parts are marked in comments).

  After this will get in, I plan to jump away from this dull military stuff
again and have a look to aicity.c probably. I just submit this because it's
origins are in one of my oldest patch and it was waiting for include already
pretty long time, so I decided to give it a try ;).

Happy hacking in new year,

-- 

                                Petr "Pasky" Baudis

UN*X programmer, UN*X administrator, hobbies = IPv6, IRC, FreeCiv, (e)links
.
The advantage of GUI is that you can see everything you can change.
The disadvantage of GUI is that you can change only what you can see.
.
Public PGP key, geekcode and stuff: http://pasky.ji.cz/~pasky/

Attachment: exploring-cleanup.patch
Description: Text document


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