From e233079d2d8dbdc1bcd7beb593f047b26a6c4cc9 Mon Sep 17 00:00:00 2001 From: Shauren Date: Mon, 24 Feb 2025 14:33:56 +0100 Subject: Core/Units: Refactor GetCastSpellInfo to return values via return value and hide internal lookup state tracker --- src/server/scripts/Spells/spell_shaman.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'src/server/scripts') diff --git a/src/server/scripts/Spells/spell_shaman.cpp b/src/server/scripts/Spells/spell_shaman.cpp index da89bd4c197..58591a25ae9 100644 --- a/src/server/scripts/Spells/spell_shaman.cpp +++ b/src/server/scripts/Spells/spell_shaman.cpp @@ -1788,10 +1788,7 @@ class spell_sha_thorims_invocation_trigger : public SpellScript if (!spellIdHolder) return; - SpellInfo const* spellInfo = sSpellMgr->GetSpellInfo(spellIdHolder->SpellIdToTrigger, GetCastDifficulty()); - TriggerCastFlags triggerFlags = TRIGGERED_NONE; - Unit::GetCastSpellInfoContext overrideContext; - spellInfo = caster->GetCastSpellInfo(spellInfo, triggerFlags, &overrideContext); + auto [spellInfo, triggerFlags] = caster->GetCastSpellInfo(sSpellMgr->GetSpellInfo(spellIdHolder->SpellIdToTrigger, GetCastDifficulty())); // Remove Overflowing Maelstrom spellmod early to make next cast behave as if it consumed only 5 or less maelstrom stacks // this works because consuming "up to 5 stacks" will always cause Maelstrom Weapon stacks to drop to 5 or lower -- cgit v1.2.3