Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2003:
[Freeciv-Dev] Re: (PR#2370) Path finding
Home

[Freeciv-Dev] Re: (PR#2370) Path finding

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxx
Cc: rt@xxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#2370) Path finding
From: "Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>
Date: Wed, 12 Feb 2003 14:07:58 -0800

On Tue, 11 Feb 2003, Raimar Falke via RT wrote:

> > Now I'm wondering, didn't you ever write any code which actually made path
> > finding work inside the game?
> 
> No. AFAIK Greg and I only used plain timing measures or used the
> warmap call to also trigger a pf call.

I adapted old explorer code to use it but then it disappeared in the 
testing / comparing stages.

> Problems:
>  - quality of path_finding.c (later)

quality of path_finding.c is superb!  Five stars!  Too good so it stands
out, so I guess that must be fixed...

>  - name of path_tools

changed to pf_tools

>  - pqueue:
>    * no comment at all what this is (also bad filename IMHO)
>    * PQDATUM* suggests a flexibility which isn't there
>    * INIT_ITEMS, *tiny* shouldn't be public

please fix

>  - get_move_costs2 and update_move_costs2 were used by my
>  implmentation. remove it.

done

>  - data/nation/Makefile.am is in the diff

removed

>  - noise in server/stdinhand.c

removed

Result attached.

G.

Attachment: pf9.diff.gz
Description: pf9.diff.gz


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