Core/Scripts: Change null-check to assert

- ToPlayer() should never be null in this context

1 files changed, 2 insertions, 4 deletions

diff --git a/src/server/scripts/Northrend/IcecrownCitadel/boss_sindragosa.cpp b/src/server/scripts/Northrend/IcecrownCitadel/boss_sindragosa.cpp
index 7d78f407ab3..9e8b19fef71 100644
--- a/src/server/scripts/Northrend/IcecrownCitadel/boss_sindragosa.cpp
+++ b/src/server/scripts/Northrend/IcecrownCitadel/boss_sindragosa.cpp
@@ -1094,10 +1094,8 @@ class spell_sindragosa_s_fury : public SpellScriptLoader
             {
                 targets.remove_if([](WorldObject* obj) -> bool
                 {
-                    if (Player* target = obj->ToPlayer())
-                        if (target->IsGameMaster())
-                            return true;
-                    return false;
+                    // SPELL_ATTR3_ONLY_TARGET_PLAYERS present on the spell, we can safely cast to Player
+                    return ASSERT_NOTNULL(obj->ToPlayer())->IsGameMaster();
                 });
 
                 _targetCount = targets.size();