Fixed all unused-parameters warnings

issue #121 used clang-tidy to achieve this
2026-01-29 00:23:48 +00:00 · 2017-09-18 14:23:26 +02:00
parent aa87ec685b
commit 2b2e299ccc
229 changed files with 643 additions and 643 deletions
--- a/src/scripts/Outland/Auchindoun/AuchenaiCrypts/boss_shirrak_the_dead_watcher.cpp
+++ b/src/scripts/Outland/Auchindoun/AuchenaiCrypts/boss_shirrak_the_dead_watcher.cpp
@@ -191,7 +191,7 @@ class spell_auchenai_possess : public SpellScriptLoader
                amplitude = 2000;
            }

-            void Update(AuraEffect* effect)
+            void Update(AuraEffect*  /*effect*/)
            {
                // Xinef: Charm is removed when target is at or below 50%hp
                if (Unit* owner = GetUnitOwner())
--- a/src/scripts/Outland/BlackTemple/boss_bloodboil.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_bloodboil.cpp
@@ -94,7 +94,7 @@ class boss_gurtogg_bloodboil : public CreatureScript
                events.ScheduleEvent(EVENT_SPELL_BERSERK, 600000);
            }

-            void KilledUnit(Unit* victim)
+            void KilledUnit(Unit*  /*victim*/)
            {
                if (events.GetNextEventTime(EVENT_KILL_TALK) == 0)
                {
--- a/src/scripts/Outland/BlackTemple/boss_illidan.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_illidan.cpp
@@ -326,7 +326,7 @@ class boss_illidan_stormrage : public CreatureScript

            void MoveInLineOfSight(Unit*) { }

-            void JustDied(Unit* killer)
+            void JustDied(Unit*  /*killer*/)
            {
                me->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NOT_SELECTABLE|UNIT_FLAG_NON_ATTACKABLE);
                summons.DespawnEntry(NPC_PARASITIC_SHADOWFIEND);
@@ -334,7 +334,7 @@ class boss_illidan_stormrage : public CreatureScript
                instance->SaveToDB();
            }

-            void KilledUnit(Unit* victim)
+            void KilledUnit(Unit*  /*victim*/)
            {
                if (events.GetNextEventTime(EVENT_KILL_TALK) == 0)
                {
@@ -791,7 +791,7 @@ class npc_akama_illidan : public CreatureScript
                summons.DespawnAll();
            }

-            void sGossipSelect(Player* player, uint32 /*sender*/, uint32 action)
+            void sGossipSelect(Player* player, uint32 /*sender*/, uint32  /*action*/)
            {
                player->CLOSE_GOSSIP_MENU();
                me->SetUInt32Value(UNIT_NPC_FLAGS, UNIT_NPC_FLAG_NONE);
@@ -1021,7 +1021,7 @@ class spell_illidan_parasitic_shadowfiend : public SpellScriptLoader
        {
            PrepareAuraScript(spell_illidan_parasitic_shadowfiend_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (!GetTarget()->HasAura(SPELL_SHADOW_PRISON) && GetTarget()->GetInstanceScript() && GetTarget()->GetInstanceScript()->IsEncounterInProgress())
                    GetTarget()->CastSpell(GetTarget(), SPELL_SUMMON_PARASITIC_SHADOWFIENDS, true);
@@ -1048,7 +1048,7 @@ class spell_illidan_parasitic_shadowfiend_trigger : public SpellScriptLoader
        {
            PrepareAuraScript(spell_illidan_parasitic_shadowfiend_trigger_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (!GetTarget()->HasAura(SPELL_SHADOW_PRISON) && GetTarget()->GetInstanceScript() && GetTarget()->GetInstanceScript()->IsEncounterInProgress())
                    GetTarget()->CastSpell(GetTarget(), SPELL_SUMMON_PARASITIC_SHADOWFIENDS, true);
@@ -1125,7 +1125,7 @@ class spell_illidan_tear_of_azzinoth_summon_channel : public SpellScriptLoader
        {
            PrepareAuraScript(spell_illidan_tear_of_azzinoth_summon_channel_AuraScript);

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                if (Unit* caster = GetCaster())
@@ -1200,7 +1200,7 @@ class spell_illidan_demon_transform1 : public SpellScriptLoader
                return GetUnitOwner()->GetTypeId() == TYPEID_UNIT;
            }

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                SetDuration(0);
@@ -1378,7 +1378,7 @@ class spell_illidan_cage_trap_stun : public SpellScriptLoader
        {
            PrepareAuraScript(spell_illidan_cage_trap_stun_AuraScript);

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                SetDuration(0);
--- a/src/scripts/Outland/BlackTemple/boss_mother_shahraz.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_mother_shahraz.cpp
@@ -222,7 +222,7 @@ class spell_mother_shahraz_saber_lash : public SpellScriptLoader
        {
            PrepareAuraScript(spell_mother_shahraz_saber_lash_AuraScript);

-            bool CheckProc(ProcEventInfo& eventInfo)
+            bool CheckProc(ProcEventInfo&  /*eventInfo*/)
            {
                return false;
            }
@@ -275,7 +275,7 @@ class spell_mother_shahraz_fatal_attraction : public SpellScriptLoader
                    }
            }

-            void HandleTeleportUnits(SpellEffIndex effIndex)
+            void HandleTeleportUnits(SpellEffIndex  /*effIndex*/)
            {
                if (Unit* target = GetHitUnit())
                    GetCaster()->CastSpell(target, SPELL_FATAL_ATTRACTION_AURA, true);
@@ -304,7 +304,7 @@ class spell_mother_shahraz_fatal_attraction_dummy : public SpellScriptLoader
        {
            PrepareSpellScript(spell_mother_shahraz_fatal_attraction_dummy_SpellScript);

-            void HandleDummy(SpellEffIndex effIndex)
+            void HandleDummy(SpellEffIndex  /*effIndex*/)
            {
                if (Unit* target = GetHitUnit())
                {
--- a/src/scripts/Outland/BlackTemple/boss_reliquary_of_souls.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_reliquary_of_souls.cpp
@@ -511,7 +511,7 @@ class boss_essence_of_anger : public CreatureScript
                }
            }

-            void JustDied(Unit* killer)
+            void JustDied(Unit*  /*killer*/)
            {
                Talk(ANGER_SAY_DEATH);
                if (me->IsSummon())
@@ -677,7 +677,7 @@ class spell_reliquary_of_souls_aura_of_desire : public SpellScriptLoader
                    amount = std::max<int32>(-100, -5*int32(effect->GetTickNumber()));
            }

-            void Update(AuraEffect const* effect)
+            void Update(AuraEffect const*  /*effect*/)
            {                
                PreventDefaultAction();
                if (AuraEffect* effect = GetAura()->GetEffect(EFFECT_1))
@@ -708,13 +708,13 @@ class spell_reliquary_of_souls_aura_of_anger : public SpellScriptLoader
        {
            PrepareAuraScript(spell_reliquary_of_souls_aura_of_anger_AuraScript);

-            void CalculateAmount(AuraEffect const* aurEff, int32& amount, bool& /*canBeRecalculated*/)
+            void CalculateAmount(AuraEffect const*  /*aurEff*/, int32& amount, bool& /*canBeRecalculated*/)
            {
                if (AuraEffect* effect = GetAura()->GetEffect(EFFECT_0))
                    amount = amount*effect->GetTickNumber();
            }

-            void Update(AuraEffect const* effect)
+            void Update(AuraEffect const*  /*effect*/)
            {
                if (AuraEffect* effect = GetAura()->GetEffect(EFFECT_0))
                    effect->RecalculateAmount();
--- a/src/scripts/Outland/BlackTemple/boss_shade_of_akama.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_shade_of_akama.cpp
@@ -553,13 +553,13 @@ class spell_shade_of_akama_shade_soul_channel : public SpellScriptLoader
        {
            PrepareAuraScript(spell_shade_of_akama_shade_soul_channel_AuraScript)
            
-            void HandleEffectApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectApply(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (Unit* caster = GetCaster())
                    caster->SetFacingToObject(GetTarget());
            }

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (Aura* aura = GetTarget()->GetAura(GetSpellInfo()->Effects[EFFECT_1].TriggerSpell))
                    aura->ModStackAmount(-1);
--- a/src/scripts/Outland/BlackTemple/boss_teron_gorefiend.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_teron_gorefiend.cpp
@@ -103,7 +103,7 @@ class boss_teron_gorefiend : public CreatureScript
                events.ScheduleEvent(EVENT_SPELL_SHADOW_OF_DEATH, 20000);
            }

-            void KilledUnit(Unit* victim)
+            void KilledUnit(Unit*  /*victim*/)
            {
                if (events.GetNextEventTime(EVENT_TALK_KILL) == 0)
                {
@@ -184,7 +184,7 @@ class spell_teron_gorefiend_shadow_of_death : public SpellScriptLoader
        {
            PrepareAuraScript(spell_teron_gorefiend_shadow_of_death_AuraScript)

-            void Absorb(AuraEffect* /*aurEff*/, DamageInfo & dmgInfo, uint32 & absorbAmount)
+            void Absorb(AuraEffect* /*aurEff*/, DamageInfo &  /*dmgInfo*/, uint32 &  /*absorbAmount*/)
            {
                PreventDefaultAction();
            }
@@ -232,7 +232,7 @@ class spell_teron_gorefiend_spirit_lance : public SpellScriptLoader
                    amount -= (amount / effect->GetTotalTicks()) * effect->GetTickNumber();
            }

-            void Update(AuraEffect const* effect)
+            void Update(AuraEffect const*  /*effect*/)
            {                
                PreventDefaultAction();
                if (AuraEffect* effect = GetAura()->GetEffect(EFFECT_1))
--- a/src/scripts/Outland/BlackTemple/boss_warlord_najentus.cpp
+++ b/src/scripts/Outland/BlackTemple/boss_warlord_najentus.cpp
@@ -137,7 +137,7 @@ class spell_najentus_needle_spine : public SpellScriptLoader
        {
            PrepareSpellScript(spell_najentus_needle_spine_SpellScript);

-            void HandleDummy(SpellEffIndex effIndex)
+            void HandleDummy(SpellEffIndex  /*effIndex*/)
            {
                if (Unit* target = GetHitUnit())
                    GetCaster()->CastSpell(target, SPELL_NEEDLE_SPINE_DAMAGE, true);
@@ -164,7 +164,7 @@ class spell_najentus_hurl_spine : public SpellScriptLoader
        {
            PrepareSpellScript(spell_najentus_hurl_spine_SpellScript);

-            void HandleSchoolDamage(SpellEffIndex effIndex)
+            void HandleSchoolDamage(SpellEffIndex  /*effIndex*/)
            {
                Unit* target = GetHitUnit();
                if (target && target->HasAura(SPELL_TIDAL_SHIELD))
--- a/src/scripts/Outland/BlackTemple/illidari_council.cpp
+++ b/src/scripts/Outland/BlackTemple/illidari_council.cpp
@@ -194,7 +194,7 @@ public:
            }
        }

-        void UpdateAI(uint32 diff)
+        void UpdateAI(uint32  /*diff*/)
        {
            if (!me->isActiveObject())
                return;
@@ -256,7 +256,7 @@ struct boss_illidari_council_memberAI : public ScriptedAI
            council->GetAI()->DoAction(ACTION_END_ENCOUNTER);
    }

-    void EnterCombat(Unit* who)
+    void EnterCombat(Unit*  /*who*/)
    {
        if (Creature* council = ObjectAccessor::GetCreature(*me, instance->GetData64(NPC_ILLIDARI_COUNCIL)))
            council->GetAI()->DoAction(ACTION_START_ENCOUNTER);
--- a/src/scripts/Outland/BlackTemple/instance_black_temple.cpp
+++ b/src/scripts/Outland/BlackTemple/instance_black_temple.cpp
@@ -379,7 +379,7 @@ class spell_black_temple_curse_of_the_bleakheart : public SpellScriptLoader
                amplitude = 5000;
            }

-            void Update(AuraEffect const* effect)
+            void Update(AuraEffect const*  /*effect*/)
            {                
                PreventDefaultAction();
                if (roll_chance_i(20))
@@ -408,7 +408,7 @@ class spell_black_temple_skeleton_shot : public SpellScriptLoader
        {
            PrepareAuraScript(spell_black_temple_skeleton_shot_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTargetApplication()->GetRemoveMode() == AURA_REMOVE_BY_DEATH)
                    GetTarget()->CastSpell(GetTarget(), GetSpellInfo()->Effects[EFFECT_2].CalcValue(), true);
@@ -435,7 +435,7 @@ class spell_black_temple_wyvern_sting : public SpellScriptLoader
        {
            PrepareAuraScript(spell_black_temple_wyvern_sting_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (Unit* caster = GetCaster())
                    caster->CastSpell(GetTarget(), SPELL_WYVERN_STING, true);
@@ -618,7 +618,7 @@ class spell_black_temple_curse_of_vitality : public SpellScriptLoader
        {
            PrepareAuraScript(spell_black_temple_curse_of_vitality_AuraScript);

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                if (GetUnitOwner()->HealthBelowPct(50))
                    SetDuration(0);
@@ -645,7 +645,7 @@ class spell_black_temple_dementia : public SpellScriptLoader
        {
            PrepareAuraScript(spell_black_temple_dementia_AuraScript);

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                if (roll_chance_i(50))
                    GetTarget()->CastSpell(GetTarget(), SPELL_DEMENTIA1, true);
--- a/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_hydross_the_unstable.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_hydross_the_unstable.cpp
@@ -281,14 +281,14 @@ class spell_hydross_cleansing_field_aura : public SpellScriptLoader
        {
            PrepareAuraScript(spell_hydross_cleansing_field_aura_AuraScript)

-            void HandleEffectApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectApply(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTarget()->GetEntry() == NPC_HYDROSS_THE_UNSTABLE)
                    if (Unit* caster = GetCaster())
                        caster->CastSpell(caster, SPELL_CLEANSING_FIELD, true);
            }

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTarget()->GetEntry() == NPC_HYDROSS_THE_UNSTABLE)
                    if (Unit* caster = GetCaster())
@@ -317,7 +317,7 @@ class spell_hydross_cleansing_field_command : public SpellScriptLoader
        {
            PrepareAuraScript(spell_hydross_cleansing_field_command_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTarget()->HasUnitState(UNIT_STATE_CASTING))
                    GetTarget()->InterruptNonMeleeSpells(false);
@@ -346,7 +346,7 @@ class spell_hydross_mark_of_hydross : public SpellScriptLoader
        {
            PrepareAuraScript(spell_hydross_mark_of_hydross_AuraScript)

-            void HandleEffectApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectApply(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                GetTarget()->RemoveAurasByType(SPELL_AURA_MOD_DAMAGE_PERCENT_TAKEN, GetCasterGUID(), GetAura());
            }
--- a/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_lady_vashj.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_lady_vashj.cpp
@@ -391,7 +391,7 @@ class spell_lady_vashj_magic_barrier : public SpellScriptLoader
        {
            PrepareAuraScript(spell_lady_vashj_magic_barrier_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                Unit::DealDamage(GetTarget(), GetTarget(), GetTarget()->CountPctFromMaxHealth(5));
            }
--- a/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_leotheras_the_blind.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_leotheras_the_blind.cpp
@@ -116,7 +116,7 @@ class boss_leotheras_the_blind : public CreatureScript
                    me->SummonCreature(NPC_GREYHEART_SPELLBINDER, channelersPos[i], TEMPSUMMON_CORPSE_TIMED_DESPAWN, 5000);
            }

-            void MoveInLineOfSight(Unit* who) { }
+            void MoveInLineOfSight(Unit*  /*who*/) { }

            void JustSummoned(Creature* summon)
            {
@@ -143,7 +143,7 @@ class boss_leotheras_the_blind : public CreatureScript
                    }
            }

-            void KilledUnit(Unit* victim)
+            void KilledUnit(Unit*  /*victim*/)
            {
                if (events.GetNextEventTime(EVENT_KILL_TALK) == 0)
                {
@@ -420,12 +420,12 @@ class spell_leotheras_insidious_whisper : public SpellScriptLoader
        {
            PrepareAuraScript(spell_leotheras_insidious_whisper_AuraScript)

-            void HandleEffectApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectApply(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                GetUnitOwner()->CastSpell(GetUnitOwner(), SPELL_SUMMON_INNER_DEMON, true);
            }

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTargetApplication()->GetRemoveMode() != AURA_REMOVE_BY_DEFAULT)
                    if (InstanceScript* instance = GetUnitOwner()->GetInstanceScript())
--- a/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_lurker_below.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SerpentShrine/boss_lurker_below.cpp
@@ -108,7 +108,7 @@ class boss_the_lurker_below : public CreatureScript
                events.ScheduleEvent(EVENT_PHASE_2, 125000);
            }

-            void MoveInLineOfSight(Unit* who)
+            void MoveInLineOfSight(Unit*  /*who*/)
            {
            }

--- a/src/scripts/Outland/CoilfangReservoir/SerpentShrine/instance_serpent_shrine.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SerpentShrine/instance_serpent_shrine.cpp
@@ -129,7 +129,7 @@ class instance_serpent_shrine : public InstanceMapScript
                return 0;
            }

-            void SetData(uint32 type, uint32 data)
+            void SetData(uint32 type, uint32  /*data*/)
            {
                switch (type)
                {
@@ -229,7 +229,7 @@ class spell_serpentshrine_cavern_serpentshrine_parasite : public SpellScriptLoad
        {
            PrepareAuraScript(spell_serpentshrine_cavern_serpentshrine_parasite_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTarget()->GetInstanceScript() && GetTarget()->GetInstanceScript()->IsEncounterInProgress())
                    GetTarget()->CastSpell(GetTarget(), SPELL_SUMMON_SERPENTSHRINE_PARASITE, true);
@@ -256,7 +256,7 @@ class spell_serpentshrine_cavern_serpentshrine_parasite_trigger : public SpellSc
        {
            PrepareAuraScript(spell_serpentshrine_cavern_serpentshrine_parasite_trigger_AuraScript)

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (GetTarget()->GetInstanceScript() && GetTarget()->GetInstanceScript()->IsEncounterInProgress())
                    GetTarget()->CastSpell(GetTarget(), SPELL_SUMMON_SERPENTSHRINE_PARASITE, true);
@@ -338,7 +338,7 @@ class spell_serpentshrine_cavern_coilfang_water : public SpellScriptLoader
        {
            PrepareAuraScript(spell_serpentshrine_cavern_coilfang_water_AuraScript)

-            void HandleEffectApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectApply(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                if (InstanceScript* instance = GetUnitOwner()->GetInstanceScript())
                    if (instance->GetBossState(DATA_THE_LURKER_BELOW) != DONE)
@@ -346,7 +346,7 @@ class spell_serpentshrine_cavern_coilfang_water : public SpellScriptLoader
                            GetUnitOwner()->CastSpell(GetUnitOwner(), SPELL_SCALDING_WATER, true);
            }

-            void HandleEffectRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
+            void HandleEffectRemove(AuraEffect const*  /*aurEff*/, AuraEffectHandleModes /*mode*/)
            {
                GetUnitOwner()->RemoveAurasDueToSpell(SPELL_SCALDING_WATER);
            }
@@ -362,7 +362,7 @@ class spell_serpentshrine_cavern_coilfang_water : public SpellScriptLoader
            }
            

-            void HandlePeriodic(AuraEffect const* aurEff)
+            void HandlePeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                InstanceScript* instance = GetUnitOwner()->GetInstanceScript();
--- a/src/scripts/Outland/CoilfangReservoir/SlavePens/boss_ahune.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/SlavePens/boss_ahune.cpp
@@ -288,7 +288,7 @@ public:
            }
        }

-        void JustDied(Unit* killer)
+        void JustDied(Unit*  /*killer*/)
        {
            summons.DespawnAll();
            me->DespawnOrUnsummon(15000);
--- a/src/scripts/Outland/CoilfangReservoir/underbog/boss_the_black_stalker.cpp
+++ b/src/scripts/Outland/CoilfangReservoir/underbog/boss_the_black_stalker.cpp
@@ -186,7 +186,7 @@ public:
            amplitude = urand(10*IN_MILLISECONDS, 200*IN_MILLISECONDS);
        }

-        void Update(AuraEffect* effect)
+        void Update(AuraEffect*  /*effect*/)
        {
            SetDuration(0);
        }
--- a/src/scripts/Outland/GruulsLair/boss_gruul.cpp
+++ b/src/scripts/Outland/GruulsLair/boss_gruul.cpp
@@ -69,7 +69,7 @@ class boss_gruul : public CreatureScript
                events.ScheduleEvent(EVENT_GROUND_SLAM, 35000);
            }

-            void KilledUnit(Unit* who)
+            void KilledUnit(Unit*  /*who*/)
            {
                if (events.GetNextEventTime(EVENT_RECENTLY_SPOKEN) == 0)
                {
--- a/src/scripts/Outland/GruulsLair/boss_high_king_maulgar.cpp
+++ b/src/scripts/Outland/GruulsLair/boss_high_king_maulgar.cpp
@@ -77,7 +77,7 @@ public:
            me->SetLootMode(0);
        }

-        void KilledUnit(Unit* victim)
+        void KilledUnit(Unit*  /*victim*/)
        {
            if (events.GetNextEventTime(EVENT_RECENTLY_SPOKEN) == 0)
            {
--- a/src/scripts/Outland/GruulsLair/instance_gruuls_lair.cpp
+++ b/src/scripts/Outland/GruulsLair/instance_gruuls_lair.cpp
@@ -101,7 +101,7 @@ class instance_gruuls_lair : public InstanceMapScript
                return true;
            }

-            void SetData(uint32 type, uint32 id)
+            void SetData(uint32 type, uint32  /*id*/)
            {
                if (type == DATA_ADDS_KILLED)
                    if (Creature* maulgar = instance->GetCreature(_maulgarGUID))
--- a/src/scripts/Outland/HellfireCitadel/BloodFurnace/boss_kelidan_the_breaker.cpp
+++ b/src/scripts/Outland/HellfireCitadel/BloodFurnace/boss_kelidan_the_breaker.cpp
@@ -87,7 +87,7 @@ class boss_kelidan_the_breaker : public CreatureScript
                    instance->SetData(DATA_KELIDAN, NOT_STARTED);
            }

-            void EnterCombat(Unit* who)
+            void EnterCombat(Unit*  /*who*/)
            {
                events.ScheduleEvent(EVENT_SPELL_VOLLEY, 1000);
                events.ScheduleEvent(EVENT_SPELL_CORRUPTION, 5000);
@@ -296,7 +296,7 @@ class npc_shadowmoon_channeler : public CreatureScript
                return NULL;
            }

-            void EnterCombat(Unit* who)
+            void EnterCombat(Unit*  /*who*/)
            {
                if (Creature* kelidan = GetKelidan())
                    kelidan->AI()->DoAction(ACTION_CHANNELER_ENGAGED);
@@ -306,7 +306,7 @@ class npc_shadowmoon_channeler : public CreatureScript
                events.ScheduleEvent(EVENT_SPELL_MARK, urand(5000, 6500));
            }

-            void JustDied(Unit* killer)
+            void JustDied(Unit*  /*killer*/)
            {
                if (Creature* kelidan = GetKelidan())
                    kelidan->AI()->DoAction(ACTION_CHANNELER_DIED);
--- a/src/scripts/Outland/HellfireCitadel/HellfireRamparts/boss_vazruden_the_herald.cpp
+++ b/src/scripts/Outland/HellfireCitadel/HellfireRamparts/boss_vazruden_the_herald.cpp
@@ -119,7 +119,7 @@ class boss_vazruden_the_herald : public CreatureScript
                    BossAI::EnterEvadeMode();
            }

-            void UpdateAI(uint32 diff)
+            void UpdateAI(uint32  /*diff*/)
            {
                if (!me->IsVisible() && summons.size() == 0)
                    BossAI::EnterEvadeMode();
--- a/src/scripts/Outland/HellfireCitadel/MagtheridonsLair/boss_magtheridon.cpp
+++ b/src/scripts/Outland/HellfireCitadel/MagtheridonsLair/boss_magtheridon.cpp
@@ -106,7 +106,7 @@ class boss_magtheridon : public CreatureScript
                me->SetFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NOT_SELECTABLE|UNIT_FLAG_IMMUNE_TO_PC);
            }

-            void KilledUnit(Unit* victim)
+            void KilledUnit(Unit*  /*victim*/)
            {
                if (events.GetNextEventTime(EVENT_RECENTLY_SPOKEN) == 0)
                {
--- a/src/scripts/Outland/HellfireCitadel/ShatteredHalls/boss_warchief_kargath_bladefist.cpp
+++ b/src/scripts/Outland/HellfireCitadel/ShatteredHalls/boss_warchief_kargath_bladefist.cpp
@@ -70,7 +70,7 @@ class boss_warchief_kargath_bladefist : public CreatureScript
                        executioner->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NON_ATTACKABLE);
            }

-            void EnterCombat(Unit* who)
+            void EnterCombat(Unit*  /*who*/)
            {
                Talk(SAY_AGGRO);
                _EnterCombat();
--- a/src/scripts/Outland/TempestKeep/Eye/boss_alar.cpp
+++ b/src/scripts/Outland/TempestKeep/Eye/boss_alar.cpp
@@ -359,7 +359,7 @@ class spell_alar_flame_quills : public SpellScriptLoader
        {
            PrepareAuraScript(spell_alar_flame_quills_AuraScript);

-            void HandlePeriodic(AuraEffect const* aurEff)
+            void HandlePeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();

--- a/src/scripts/Outland/TempestKeep/Eye/boss_kaelthas.cpp
+++ b/src/scripts/Outland/TempestKeep/Eye/boss_kaelthas.cpp
@@ -750,7 +750,7 @@ class spell_kaelthas_remote_toy : public SpellScriptLoader
        {
            PrepareAuraScript(spell_kaelthas_remote_toy_AuraScript);

-            void HandlePeriodic(AuraEffect const* aurEff)
+            void HandlePeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                if (roll_chance_i(66))
@@ -859,7 +859,7 @@ class spell_kaelthas_burn : public SpellScriptLoader
        {
            PrepareAuraScript(spell_kaelthas_burn_AuraScript);

-            void HandlePeriodic(AuraEffect const* aurEff)
+            void HandlePeriodic(AuraEffect const*  /*aurEff*/)
            {
                Unit::DealDamage(GetUnitOwner(), GetUnitOwner(), GetUnitOwner()->CountPctFromMaxHealth(5)+1);
            }
--- a/src/scripts/Outland/TempestKeep/Eye/instance_the_eye.cpp
+++ b/src/scripts/Outland/TempestKeep/Eye/instance_the_eye.cpp
@@ -156,7 +156,7 @@ class spell_the_eye_countercharge : public SpellScriptLoader
        {
            PrepareAuraScript(spell_the_eye_counterchargeScript);

-            bool PrepareProc(ProcEventInfo& eventInfo)
+            bool PrepareProc(ProcEventInfo&  /*eventInfo*/)
            {
                // xinef: prevent charge drop
                PreventDefaultAction();
--- a/src/scripts/Outland/TempestKeep/Mechanar/boss_nethermancer_sepethrea.cpp
+++ b/src/scripts/Outland/TempestKeep/Mechanar/boss_nethermancer_sepethrea.cpp
@@ -38,7 +38,7 @@ class boss_nethermancer_sepethrea : public CreatureScript
        {
            boss_nethermancer_sepethreaAI(Creature* creature) : BossAI(creature, DATA_NETHERMANCER_SEPRETHREA) { }

-            void EnterCombat(Unit* who)
+            void EnterCombat(Unit*  /*who*/)
            {
                _EnterCombat();
                events.ScheduleEvent(EVENT_FROST_ATTACK, 6000);
--- a/src/scripts/Outland/TempestKeep/Mechanar/boss_pathaleon_the_calculator.cpp
+++ b/src/scripts/Outland/TempestKeep/Mechanar/boss_pathaleon_the_calculator.cpp
@@ -56,7 +56,7 @@ class boss_pathaleon_the_calculator : public CreatureScript
                me->SetReactState(REACT_PASSIVE);
            }

-            void DoAction(int32 param)
+            void DoAction(int32  /*param*/)
            {
                me->SetVisible(true);
                me->CastSpell(me, SPELL_TELEPORT_VISUAL, true);
--- a/src/scripts/Outland/TempestKeep/arcatraz/instance_arcatraz.cpp
+++ b/src/scripts/Outland/TempestKeep/arcatraz/instance_arcatraz.cpp
@@ -110,7 +110,7 @@ class instance_arcatraz : public InstanceMapScript
                }
            }

-            uint32 GetData(uint32 type) const
+            uint32 GetData(uint32  /*type*/) const
            {
                return 0;
            }
--- a/src/scripts/Outland/zone_blades_edge_mountains.cpp
+++ b/src/scripts/Outland/zone_blades_edge_mountains.cpp
@@ -113,7 +113,7 @@ class spell_npc22275_crystal_prison : public SpellScriptLoader
        {
            PrepareAuraScript(spell_npc22275_crystal_prison_AuraScript);

-            void OnPeriodic(AuraEffect const* aurEff)
+            void OnPeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                SetDuration(0);
--- a/src/scripts/Outland/zone_nagrand.cpp
+++ b/src/scripts/Outland/zone_nagrand.cpp
@@ -424,7 +424,7 @@ public:
        uint32 HealTimer;
        uint32 FrostShockTimer;

-        void SetGUID(uint64 guid, int32 questId)
+        void SetGUID(uint64 guid, int32  /*questId*/)
        {
            me->SetStandState(UNIT_STAND_STATE_STAND);
            Start(true, false, guid);
--- a/src/scripts/Outland/zone_shadowmoon_valley.cpp
+++ b/src/scripts/Outland/zone_shadowmoon_valley.cpp
@@ -45,7 +45,7 @@ class spell_q10612_10613_the_fel_and_the_furious : public SpellScriptLoader
        {
            PrepareSpellScript(spell_q10612_10613_the_fel_and_the_furious_SpellScript);

-            void HandleScriptEffect(SpellEffIndex effIndex)
+            void HandleScriptEffect(SpellEffIndex  /*effIndex*/)
            {
                Player* charmer = GetCaster()->GetCharmerOrOwnerPlayerOrPlayerItself();
                if (!charmer)
@@ -251,7 +251,7 @@ public:
            }
        }

-        void UpdateAI(uint32 diff)
+        void UpdateAI(uint32  /*diff*/)
        {
            if (!UpdateVictim())
                return;
--- a/src/scripts/Outland/zone_terokkar_forest.cpp
+++ b/src/scripts/Outland/zone_terokkar_forest.cpp
@@ -176,7 +176,7 @@ class spell_q10036_torgos : public SpellScriptLoader
        {
            PrepareSpellScript(spell_q10036_torgos_SpellScript);

-            void HandleSendEvent(SpellEffIndex effIndex)
+            void HandleSendEvent(SpellEffIndex  /*effIndex*/)
            {
                if (Creature* torgos = GetCaster()->FindNearestCreature(NPC_TORGOS, 100.0f, true))
                    torgos->GetAI()->AttackStart(GetCaster());
@@ -212,7 +212,7 @@ class spell_q10923_evil_draws_near_summon : public SpellScriptLoader
        {
            PrepareSpellScript(spell_q10923_evil_draws_near_summon_SpellScript);

-            void HandleSendEvent(SpellEffIndex effIndex)
+            void HandleSendEvent(SpellEffIndex  /*effIndex*/)
            {
                if (Creature* auchenai = GetCaster()->FindNearestCreature(NPC_AUCHENAI_DEATH_SPIRIT, 10.0f, true))
                    auchenai->CastSpell(auchenai, SPELL_DUSTIN_UNDEAD_DRAGON_VISUAL_AURA, true);
@@ -239,7 +239,7 @@ class spell_q10923_evil_draws_near_periodic : public SpellScriptLoader
        {
            PrepareAuraScript(spell_q10923_evil_draws_near_periodic_AuraScript);

-            void HandlePeriodic(AuraEffect const* aurEff)
+            void HandlePeriodic(AuraEffect const*  /*aurEff*/)
            {
                PreventDefaultAction();
                GetUnitOwner()->CastSpell(GetUnitOwner(), RAND(SPELL_DUSTIN_UNDEAD_DRAGON_VISUAL1, SPELL_DUSTIN_UNDEAD_DRAGON_VISUAL2), true);
--- a/src/scripts/Outland/zone_zangarmarsh.cpp
+++ b/src/scripts/Outland/zone_zangarmarsh.cpp
@@ -47,7 +47,7 @@ public:
        return true;
    }

-    bool OnGossipSelect(Player* player, Creature* creature, uint32 /*sender*/, uint32 action)
+    bool OnGossipSelect(Player* player, Creature* creature, uint32 /*sender*/, uint32  /*action*/)
    {
        player->PlayerTalkClass->ClearMenus();
        player->CLOSE_GOSSIP_MENU();