Code review comment for lp:~widelands-dev/widelands/autosave_on_objectives

Revision history for this message
cghislai (charlyghislain) wrote :

I'm a bit lost in the lua tests. It appears I cannot make it run even with trunk code :
"could not load "test/lua/persistence.wmf"

Could you check it? I will try to make a save-in-corountine test afterwards.

I refactored m_allow_autosaving to m_allow_saving, because i guess we have to prevent manual saves as well then, right?

I removed some comments i grasped, and left some that may still be useful

« Back to merge proposal