Hi everybody,

i am reworking the code for savegames, mainly to achieve:
- there is always a "full" save (that is no snapshot only)
- there is an autosave at the beginning of this and last turn

I did a short analysis of the current code and what appears to be a mess at
first glance shows at least some structure. Still i think there is room for
improvement and while i am at it i would like to take the chance to do so.
Attached is a pdf, that shows the various links between savegame code. The
green boxes are not critical, they are supposed to be integrated with the
corresponding turn_info elements later (when i merge play_level and
replay_controller). IMO it is the yellow boxes that either should be
reworked or at least be moved to a more appropriate place.

The boxes show where the code is placed, the parameters when calling it and
a short explanation what it does (as i understood it). One box shows a "?"
there. I am not sure if this code is really needed or when it is called.
Maybe someone more familiar with the code can comment on this.

Please give me some feedback what you think about this. I also will be
available on irc tomorrow from about 8 pm (GMT+1).

Greetings

Yogi

Attachment: Wesnoth SaveGames.pdf
Description: Adobe PDF document

Reply via email to