[Freeciv-Dev] Re: (PR#7259) new function tile_has_river

["rwetmore@xxxxxxxxxxxx" <rwetmore@xxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=7259 >


Jason Short wrote:
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=7259 >
> 
> Vasco Alexandre da Silva Costa wrote:
[...]
>>Remember, Freeciv used to just emulate Civ1. I assume the T_RIVER flag is
>>to allow reading of old Freeciv scenarios.

The T_RIVER tile type was the original river implementation.

> All savegames will be affected.  Per's hack to replace T_RIVER with 
> T_GRASSLAND + S_RIVER is good, but if the terrain came entirely from the 
> ruleset there wouldn't _be_ any T_RIVER and this would be impossible.
> 
> Generally, Freeciv is backward-compatible with its savegames.  But a 
> (significant) change to the ruleset breaks savegame compatability.  Is 
> there anything we can do about this?
> 
> jason

There is nothing to stop one from reading (and writing) Civ1 compatible
savegames while generally eliminating T_RIVER from all internal
implementation code. The only real issue is one of effort, I'm pretty sure.

Cheers,
RossW
=====