URL:
<http://gna.org/patch/?3140>
Summary: Roads order to savegame
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 18 Feb 2012 02:36:05 AM EET
Category: general
Priority: 5 - Normal
Status: Ready For Test
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release: 2.5.0
_______________________________________________________
Details:
This patchs adds saving and loading of Roads order. There's not yet anything
that would be saved dependant on that order, but I found out that having this
one separately first makes development of all the other roads related savegame
handling patches much easier.
I actually would appreciate some input about how we should handle savegame
compatibility issues in development. It would be very hard to create one
megalopatch that would switch everything to use generic roads, with proper
backward compatibility hooks, at one go. Having several intermediate versions
would mean more complicated compatibility handling. Can we just ignore the
compatibility issues between two savegame formats that are introduced during
active development (trunk) of the new version? If not, do we icrease format
version number between the intermediate formats and provide compatibility
hooks, or do we adjust to such changes within main loading code?
Looking the kind of format changes required compared to older freeciv
versions, I'm currently not sure if this new style compatibility hooks system
is really suitable for the work. Hooks between earlier versions have simply
added or deleted entries from section file, but now content of one entry
affects how contents of other entries should be modified, and all this in deep
loops.
_______________________________________________________
File Attachments:
-------------------------------------------------------
Date: Sat 18 Feb 2012 02:36:05 AM EET Name: RoadOrderSave.diff Size: 4kB
By: cazfi
<http://gna.org/patch/download.php?file_id=15161>
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?3140>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
