[Asserts] New MapPersistentState system and some crashes.

[Undergarun]

mangos-worldd: /root/mangosdev/src/game/MapPersistentStateMgr.cpp:772: void MapPersistentStateManager::_ResetInstance(uint32, uint32): Assertion `"iMap->IsDungeon()" && 0' failed.

http://pastebin.com/7Z8AgDtS

http://pastebin.com/ZuYTpYZe

Custom repo, mtmaps, latest ytdb and etc... but i don't think is related to this.

[darkstalker]

compile your core with flags "-DNDEBUG" so it doesn't crash on asserts

[Undergarun]

compile your core with flags "-DNDEBUG" so it doesn't crash on asserts

Or comment the assert or... but is not the real solution, is another tempfix. Asserts are used for debug purposes so i reported this to know that in some cases, iMap->IsDungeon() is true.

[VladimirMangos]

Hmm, resets must be orginal added only for dungeon maps. I will add more strict check at loading. Maybe some outdated data including bg/arenas. In past exist possibility save BG/arenas as instances.

[added]Test with [11226]

[Undergarun]

Still crashing...

[lecails]

Still crashing...

Have you try whith a clean repository, I think mtmaps can interact whith this.

[VladimirMangos]

In any case i add checks in all possible cases, So i not see any way for get wrong data except corruption memory like weel known as possible memory corrupt things as mtmaps, as pointed above.