[Freeciv-Dev] Re: (PR#3928) Convert client to use PF

["Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>]

On Fri, 2 May 2003, Raimar Falke wrote:

> On Sun, Apr 27, 2003 at 02:48:14PM -0700, Gregory Berkolaiko wrote:
> > Yep, found a couple of bugs, thanks.
> 
> Not a bug per se but something which should be changed IMHO:
> 
> 2: PF:   35/47: (47, 2) dir=NE cost=102 (11, 6) EC=34
> 2: PF:   36/47: (48, 1) dir=SE cost=105 (11, 3) EC=35
> 2: PF:   37/47: (48, 1) dir=[Undef] cost=108 (12, 9) EC=35
> 2: PF:   38/47: (49, 2) dir=SE cost=111 (12, 6) EC=36
> 2: PF:   39/47: (50, 3) dir=SE cost=114 (12, 3) EC=37
> 
> Here 36's dir should be undef and 37's should be SE because we first
> wait and go then SE.

Yes.  I also get dodgy costs (I guess you don't cos your trireme is on 
full movepoints).  Fixed both.

Unfortunately your code is build around my buggy code so I get:
civclient: goto.c:189: update_last_part: Assertion `prev && (a->x == 
prev->x && a->y == prev->y)' failed

prev shouldbe changed to next of course.

Anyway, I willsendthe patch tomorrow.

G.