[Freeciv-Dev] Re: The missing patch

["mateusz stefek" <matusik_s@xxxxx>]

>> For ai cross-country-rail
>
>n the function
>
>static int cross_country_rail(struct unit* punit, struct player* pplayer)
>
>int d = warmap.cost[pcity->x][pcity->y];
>    if (d == 0) is_connected = 1;else
>
>You cannot assume railroad_move cost is always going to equal 0.
>Make the test if (d == railroad_move_cost). In fact, for a non zero
>movecost this function would have to be rewritten. You need a check
>that counts the number of tiles between pctiy x and y. Multiply the
>result by railroad_move_cost. If d = the result then is connected.
>
That isn't always right way (something could be smaller than MOVE_COST_RAIL)

What about adding
+#if MOVE_COST_RAIL == 0
everywhere cross_country_rail is used? (4 places)
Or should I write something more complicated but prettier?

------
mateusz stefek