We have some bugs that I think point to serious problems in the
save/load code:
#11842 Animations getting mixed up on advancement
#11612 Krash inexplicably disappears from recall list
#11556 Replays still corrupt
Just before I had to disappear and work on Emacs, I took a look at the
save code with a view to refactoring it. This waa because I need to
understand the code in order to do what I want to do with the savefile
UI (that is, group saves into game threads and make the UI browse
those game threads).
Unfortunately, that code as it is now is so complex that I could not
figure out the right place to put my thread-ID hooks. It is also so
fragile and messy that I don't actually find any of the bugs I cited
above surprising. The wonder is that we don't have more problems than
we do.
The save/load code needs a serious rewrite and cleanup; I think this
is actually our most serious structural issue in the 1.5 cycle..
Mordante had it on his to-do list at one time, but the widget code is
eating all his bandwidth. I'd do it, but I am still embroiled in two other
projects; my availability will be limited for the foreseeable future.
YogiHH, can you come back and do serious work on this? Is there
anyone else looking for a meaty, important project to do?
--
<a href="http://www.catb.org/~esr/";>Eric S. Raymond</a>
Where rights secured by the Constitution are involved, there can be no
rule making or legislation which would abrogate them.
-- Miranda vs. Arizona, 384 US 436 p. 491
_______________________________________________
Wesnoth-dev mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-dev
