Mirror/TrinityCore
1
0
Fork 0
mirror of https://github.com/TrinityCore/TrinityCore.git synced 2026-01-16 15:40:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

Core/Scripting: Fix an assertion which was triggered at lazy unloading

Browse Source 
* Also fixes 2 warnings
This commit is contained in:
Naios
2016-04-12 16:52:29 +02:00
parent 410cf0dd05
commit e82a934387
2 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/server/game/Scripting/ScriptMgr.cpp
View File

@@ -166,7 +166,8 @@ class ScriptRegistryCompositum
public:
void SetScriptNameInContext(std::string const& scriptname, std::string const& context)
{
ASSERT(_scriptnames_to_context.find(scriptname) == _scriptnames_to_context.end());
ASSERT(_scriptnames_to_context.find(scriptname) == _scriptnames_to_context.end(),
"Scriptname was assigned to this context already!");
_scriptnames_to_context.insert(std::make_pair(scriptname, context));
}
@@ -180,15 +181,18 @@ public:
void ReleaseContext(std::string const& context) final override
{
for (auto const registry : _registries)
registry->ReleaseContext(context);
// Clear the script names in context after calling the release hooks
// since it's possible that new references to a shared library
// are acquired when releasing.
for (auto itr = _scriptnames_to_context.begin();
itr != _scriptnames_to_context.end();)
if (itr->second == context)
itr = _scriptnames_to_context.erase(itr);
else
++itr;
for (auto const registry : _registries)
registry->ReleaseContext(context);
}
void SwapContext(bool initialize) final override
@@ -604,7 +608,7 @@ public:
swapped = true;
}
void BeforeSwapContext(bool initialize) override
void BeforeSwapContext(bool /*initialize*/) override
{
swapped = false;
}
@@ -634,7 +638,7 @@ public:
swapped = true;
}
void BeforeSwapContext(bool initialize) override
void BeforeSwapContext(bool /*initialize*/) override
{
if (swapped)
{