[Freeciv-Dev] Re: find_the_shortest_path() cleanup

[Jeff Mallatt <jjm@xxxxxxxxxxxx>]

At 2000/05/01 17:59 , Thue Janus Kristensen wrote:
>Some cleanup; handle airunits seperately to avoid severel problems. I
>would have done more, but some of that code is opaque.

Great!  The case is much nicer!

A few alterations.  I changed a couple of comments to be more clear (at
least to me).  Put GOTO_MOVE_STRAIGHTEST back into AIR_ case, because it
has a specific meaning which just letting the path planner run may not
replicate.  Added freelog() calls to SEA_ and AIR_ cases, like are in LAND_
case (these are missing from current code, but should be there).  Added a
default to the case, which logs and aborts.

I've run some quick tests, and it all seems to work.

goto_works-1.diff
Description: Text document

jjm