Complete.Org: Mailing Lists: Archives: freeciv-ai: March 2003:
[freeciv-ai] Re: [Freeciv-Dev] (PR#3689) server/settler.c
Home

[freeciv-ai] Re: [Freeciv-Dev] (PR#3689) server/settler.c

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: ue80@xxxxxxxxxxxxxxxxxxxxx
Subject: [freeciv-ai] Re: [Freeciv-Dev] (PR#3689) server/settler.c
From: "Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>
Date: Tue, 11 Mar 2003 06:50:12 -0800
Reply-to: rt@xxxxxxxxxxxxxx

On Tue, 11 Mar 2003, Per I. Mathisen wrote:

> On Tue, 11 Mar 2003, Gregory Berkolaiko wrote:
> > I guess the best way forward would be to first clean the routine up
> > (I believe we have something to learn from it) and then try to formulate a
> > better (simpler) solution.Or just polish the existing one.
> >
> > Any takers for the cleaning job?
> 
> I've tried several times, since my last posted cleanup of this file. I
> gave up each time - the code is just so bad, I started rewriting it from

Mhhh, the code looks relatively transperent to me, but then I haven't 
tried rewriting it...

> scratch instead. I used virtual cities, which allows for more code reuse
> and avoids much of the stupid second-guessing of other parts of the code
> which is done in settlers.c now.
> 
> I have a funky new settlers code lying about here, which is way better
> than the existing code, but it has no support for ferries or terrain
> improvement yet. It is on hold until the former has been sorted out. And
> ferries await pf.

And PF awaits you.  I am finished with it.

> Let me know if you want my preliminary code. It is currently broken
> against current cvs, but it shouldn't be too much work to update it.

Yes, send it to RT.

G.





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