[Freeciv-Dev] Re: (PR#18441) [Patch] Better warmap caching

["Marko Lindqvist" <cazfi74@xxxxxxxxx>]

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=18441 >

Marko Lindqvist wrote:
> 
>   Also, there is still two users for gotohand.c:do_unit_goto().
> grep -r do_unit_goto server/*.c

  do_unit_goto() is only place that needs find_the_shortest_path(). 
find_the_shortest_path() implements its own warmap filling method 
distinct from generate_warmap(), in theory requiring maintenance 
separately from generate_warmap() (as find_the_shortest_path() is not 
fatally broken, I'm not fixing it). Getting rid of 
find_the_shortest_path() would be good.


  - ML