diff options
| author | Shauren <shauren.trinity@gmail.com> | 2016-06-03 16:45:01 +0200 |
|---|---|---|
| committer | Shauren <shauren.trinity@gmail.com> | 2016-06-03 16:45:01 +0200 |
| commit | 3ab3a667ed8640edf67904a204bb4758b4f5e7b1 (patch) | |
| tree | 203b314723c5639b25607357d3d9c647514cd649 /src/server/game/Spells/SpellMgr.cpp | |
| parent | 7f21c2979e1bea269bb0da24ca4772b39b9771ab (diff) | |
Core/Spells: Add assertions validating various spell enums
Diffstat (limited to 'src/server/game/Spells/SpellMgr.cpp')
| -rw-r--r-- | src/server/game/Spells/SpellMgr.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/server/game/Spells/SpellMgr.cpp b/src/server/game/Spells/SpellMgr.cpp index a7ad0e55d5b..84cbbe90b7e 100644 --- a/src/server/game/Spells/SpellMgr.cpp +++ b/src/server/game/Spells/SpellMgr.cpp @@ -2697,11 +2697,11 @@ void SpellMgr::LoadSpellInfoStore() for (SpellEffectEntry const* effect : sSpellEffectStore) { - if (effect->EffectIndex >= MAX_SPELL_EFFECTS) - { - TC_LOG_ERROR("server.loading", "Spell %u has invalid EffectIndex %u, max is %u, skipped", effect->SpellID, effect->EffectIndex, uint32(MAX_SPELL_EFFECTS)); - continue; - } + ASSERT(effect->EffectIndex < MAX_SPELL_EFFECTS, "MAX_SPELL_EFFECTS must be at least %u", effect->EffectIndex); + ASSERT(effect->Effect < TOTAL_SPELL_EFFECTS, "TOTAL_SPELL_EFFECTS must be at least %u", effect->Effect); + ASSERT(effect->EffectAura < TOTAL_AURAS, "TOTAL_AURAS must be at least %u", effect->EffectAura); + ASSERT(effect->ImplicitTarget[0] < TOTAL_SPELL_TARGETS, "TOTAL_SPELL_TARGETS must be at least %u", effect->ImplicitTarget[0]); + ASSERT(effect->ImplicitTarget[1] < TOTAL_SPELL_TARGETS, "TOTAL_SPELL_TARGETS must be at least %u", effect->ImplicitTarget[1]); SpellEffectEntryVector& effectsForDifficulty = effectsBySpell[effect->SpellID][effect->DifficultyID]; if (effectsForDifficulty.size() <= effect->EffectIndex) |