The International Simutrans Forum

The new Simutrans Experimental 7.2 should read all savegames save with the official Experimental 7.1 and older versions.
Unfortunatelly it is not compatible with nightly 7.2 builds dated before 15th of March due to a version clash with standard simutrans.

If your savegames are Exp 7.1 or earlier, please upload/post one or two.

@neroden
I revised you patch. It is a bit too drastic. If you load an older savegame having stored replacements, the stored values must be read, but ignored.

If the replace code is invoked otherwise than by the disabled GUI, there is indeed a bug that needs fixing in any event.

As to "delete this", the C++ FAQ (to which I linked) was clear that this was acceptable coding in certain conditions, which I believe were fulfilled. The system is that, whenever a particular convoy has finished with the replace data object that it is using, it calls "decrement_convoys()" on the replace data object, rather than delete it, as there may be other convoys sharing that object. If it is the last convoy to use it, the replace data object deletes itself. How else would this function be accomplished as efficiently without delete this;?

The loading would only cause a convoy replacer instance to be created if the convoy was in the process of being replaced in the pre-7.2 version, I think.

As to "delete this" - does the backtrace destroying happen in MSVC++? I haven't had that problem, I don't think; and, even if it is a problem, one can just set a conditional breakpoint on the "delete this" line to find it...