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 Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev