aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rwxr-xr-xsrc/server/game/Battlegrounds/Battleground.cpp4
-rw-r--r--src/server/game/Chat/ChatLink.cpp4
-rwxr-xr-xsrc/server/game/Entities/Creature/Creature.cpp10
-rwxr-xr-xsrc/server/game/Entities/Player/Player.cpp19
-rwxr-xr-xsrc/server/game/Entities/Totem/Totem.cpp2
-rwxr-xr-xsrc/server/game/Events/GameEventMgr.cpp2
-rwxr-xr-xsrc/server/game/Globals/ObjectMgr.cpp8
-rwxr-xr-xsrc/server/game/Globals/ObjectMgr.h2
-rwxr-xr-xsrc/server/game/Spells/Auras/SpellAuraEffects.cpp7
-rwxr-xr-xsrc/server/game/Tickets/TicketMgr.cpp5
-rwxr-xr-xsrc/server/game/Tickets/TicketMgr.h2
-rw-r--r--src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/culling_of_stratholme.cpp2
-rw-r--r--src/server/scripts/Kalimdor/bloodmyst_isle.cpp9
-rw-r--r--src/server/scripts/Northrend/Nexus/EyeOfEternity/boss_malygos.cpp55
-rw-r--r--src/server/scripts/Northrend/Ulduar/ulduar/boss_razorscale.cpp4
-rw-r--r--src/server/scripts/Outland/nagrand.cpp6
16 files changed, 68 insertions, 73 deletions
diff --git a/src/server/game/Battlegrounds/Battleground.cpp b/src/server/game/Battlegrounds/Battleground.cpp
index 13e08d50d40..45308497077 100755
--- a/src/server/game/Battlegrounds/Battleground.cpp
+++ b/src/server/game/Battlegrounds/Battleground.cpp
@@ -539,8 +539,8 @@ inline Player* Battleground::_GetPlayer(BattlegroundPlayerMap::const_iterator it
inline Player* Battleground::_GetPlayerForTeam(uint32 teamId, BattlegroundPlayerMap::const_iterator itr, const char* context) const
{
- Player* player = NULL;
- if (player = _GetPlayer(itr, context))
+ Player* player = _GetPlayer(itr, context);
+ if (player)
{
uint32 team = itr->second.Team;
if (!team)
diff --git a/src/server/game/Chat/ChatLink.cpp b/src/server/game/Chat/ChatLink.cpp
index 7bf56ee85bc..218c4574639 100644
--- a/src/server/game/Chat/ChatLink.cpp
+++ b/src/server/game/Chat/ChatLink.cpp
@@ -84,7 +84,7 @@ inline bool ReadHex(std::istringstream& iss, uint32& res, uint32 length)
{
std::istringstream::pos_type pos = iss.tellg();
iss >> std::hex >> res;
- uint32 size = uint32(iss.gcount());
+ //uint32 size = uint32(iss.gcount());
if (length && uint32(iss.tellg() - pos) != length)
return false;
return !iss.fail() && !iss.eof();
@@ -93,7 +93,7 @@ inline bool ReadHex(std::istringstream& iss, uint32& res, uint32 length)
#define DELIMITER ':'
#define PIPE_CHAR '|'
-bool ChatLink::ValidateName(char* buffer, const char* context)
+bool ChatLink::ValidateName(char* buffer, const char* /*context*/)
{
_name = buffer;
return true;
diff --git a/src/server/game/Entities/Creature/Creature.cpp b/src/server/game/Entities/Creature/Creature.cpp
index 9e7aa5871ef..28b8753b2aa 100755
--- a/src/server/game/Entities/Creature/Creature.cpp
+++ b/src/server/game/Entities/Creature/Creature.cpp
@@ -301,7 +301,7 @@ bool Creature::InitEntry(uint32 Entry, uint32 /*team*/, const CreatureData *data
}
uint32 displayID = sObjectMgr->ChooseDisplayId(0, GetCreatureInfo(), data);
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(&displayID);
if (!minfo) // Cancel load if no model defined
{
sLog->outErrorDb("Creature (Entry: %u) has no model defined in table `creature_template`, can't load. ", Entry);
@@ -779,8 +779,8 @@ bool Creature::Create(uint32 guidlow, Map *map, uint32 phaseMask, uint32 Entry,
}
LoadCreaturesAddon();
- uint32 displayID = 0;
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID = GetNativeDisplayId());
+ uint32 displayID = GetNativeDisplayId();
+ CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(&displayID);
if (minfo && !isTotem()) // Cancel load if no model defined or if totem
{
SetDisplayId(displayID);
@@ -1618,8 +1618,8 @@ void Creature::Respawn(bool force)
else
setDeathState(JUST_ALIVED);
- uint32 displayID = 0;
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID = GetNativeDisplayId());
+ uint32 displayID = GetNativeDisplayId();
+ CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(&displayID);
if (minfo) // Cancel load if no model defined
{
SetDisplayId(displayID);
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp
index a3de9c30321..46325984ad2 100755
--- a/src/server/game/Entities/Player/Player.cpp
+++ b/src/server/game/Entities/Player/Player.cpp
@@ -11150,7 +11150,6 @@ InventoryResult Player::CanStoreItems(Item** pItems, int count) const
if (pItem->IsBindedNotWith(this))
return EQUIP_ERR_DONT_OWN_THAT_ITEM;
- Bag *pBag;
ItemTemplate const *pBagProto;
// item is 'one item only'
@@ -11201,11 +11200,11 @@ InventoryResult Player::CanStoreItems(Item** pItems, int count) const
for (int t = INVENTORY_SLOT_BAG_START; !b_found && t < INVENTORY_SLOT_BAG_END; ++t)
{
- if (pBag = GetBagByPos(t))
+ if (Bag* bag = GetBagByPos(t))
{
- if (ItemCanGoIntoBag(pItem->GetTemplate(), pBag->GetTemplate()))
+ if (ItemCanGoIntoBag(pItem->GetTemplate(), bag->GetTemplate()))
{
- for (uint32 j = 0; j < pBag->GetBagSize(); j++)
+ for (uint32 j = 0; j < bag->GetBagSize(); j++)
{
pItem2 = GetItemByPos(t, j);
if( pItem2 && pItem2->CanBeMergedPartlyWith(pProto) == EQUIP_ERR_OK && inv_bags[t-INVENTORY_SLOT_BAG_START][j] + pItem->GetCount() <= pProto->GetMaxStackSize())
@@ -11258,15 +11257,15 @@ InventoryResult Player::CanStoreItems(Item** pItems, int count) const
for (int t = INVENTORY_SLOT_BAG_START; !b_found && t < INVENTORY_SLOT_BAG_END; ++t)
{
- if (pBag = GetBagByPos(t))
+ if (Bag* bag = GetBagByPos(t))
{
- pBagProto = pBag->GetTemplate();
+ pBagProto = bag->GetTemplate();
// not plain container check
if (pBagProto && (pBagProto->Class != ITEM_CLASS_CONTAINER || pBagProto->SubClass != ITEM_SUBCLASS_CONTAINER) &&
ItemCanGoIntoBag(pProto, pBagProto))
{
- for (uint32 j = 0; j < pBag->GetBagSize(); j++)
+ for (uint32 j = 0; j < bag->GetBagSize(); j++)
{
if (inv_bags[t-INVENTORY_SLOT_BAG_START][j] == 0)
{
@@ -11297,15 +11296,15 @@ InventoryResult Player::CanStoreItems(Item** pItems, int count) const
// search free slot in bags
for (int t = INVENTORY_SLOT_BAG_START; !b_found && t < INVENTORY_SLOT_BAG_END; ++t)
{
- if (pBag = GetBagByPos(t))
+ if (Bag* bag = GetBagByPos(t))
{
- pBagProto = pBag->GetTemplate();
+ pBagProto = bag->GetTemplate();
// special bag already checked
if (pBagProto && (pBagProto->Class != ITEM_CLASS_CONTAINER || pBagProto->SubClass != ITEM_SUBCLASS_CONTAINER))
continue;
- for (uint32 j = 0; j < pBag->GetBagSize(); j++)
+ for (uint32 j = 0; j < bag->GetBagSize(); j++)
{
if (inv_bags[t-INVENTORY_SLOT_BAG_START][j] == 0)
{
diff --git a/src/server/game/Entities/Totem/Totem.cpp b/src/server/game/Entities/Totem/Totem.cpp
index 793b9119dfe..b216d3e344c 100755
--- a/src/server/game/Entities/Totem/Totem.cpp
+++ b/src/server/game/Entities/Totem/Totem.cpp
@@ -58,7 +58,7 @@ void Totem::InitStats(uint32 duration)
if (m_owner->GetTypeId() == TYPEID_PLAYER && cinfo)
{
uint32 displayID = sObjectMgr->ChooseDisplayId(m_owner->ToPlayer()->GetTeam(), cinfo);
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ sObjectMgr->GetCreatureModelRandomGender(&displayID);
switch (m_owner->ToPlayer()->GetTeam())
{
case ALLIANCE:
diff --git a/src/server/game/Events/GameEventMgr.cpp b/src/server/game/Events/GameEventMgr.cpp
index 004dd9d5e2d..582a389d1fb 100755
--- a/src/server/game/Events/GameEventMgr.cpp
+++ b/src/server/game/Events/GameEventMgr.cpp
@@ -1332,7 +1332,7 @@ void GameEventMgr::ChangeEquipOrModel(int16 event_id, bool activate)
{
CreatureTemplate const *cinfo = sObjectMgr->GetCreatureTemplate(data2->id);
uint32 displayID = sObjectMgr->ChooseDisplayId(0, cinfo, data2);
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ sObjectMgr->GetCreatureModelRandomGender(&displayID);
if (data2->equipmentId == 0)
itr->second.equipement_id_prev = cinfo->equipmentId;
diff --git a/src/server/game/Globals/ObjectMgr.cpp b/src/server/game/Globals/ObjectMgr.cpp
index 9a2f078de15..bac8466cec2 100755
--- a/src/server/game/Globals/ObjectMgr.cpp
+++ b/src/server/game/Globals/ObjectMgr.cpp
@@ -1122,9 +1122,9 @@ void ObjectMgr::ChooseCreatureFlags(const CreatureTemplate *cinfo, uint32& npcfl
}
}
-CreatureModelInfo const* ObjectMgr::GetCreatureModelRandomGender(uint32 &displayID)
+CreatureModelInfo const* ObjectMgr::GetCreatureModelRandomGender(uint32* displayID)
{
- CreatureModelInfo const* minfo = GetCreatureModelInfo(displayID);
+ CreatureModelInfo const* minfo = GetCreatureModelInfo(*displayID);
if (!minfo)
return NULL;
@@ -1139,7 +1139,7 @@ CreatureModelInfo const* ObjectMgr::GetCreatureModelRandomGender(uint32 &display
else
{
// Model ID changed
- displayID = minfo->modelid_other_gender;
+ *displayID = minfo->modelid_other_gender;
return minfo_tmp;
}
}
@@ -5788,7 +5788,7 @@ uint32 ObjectMgr::GetTaxiMountDisplayId(uint32 id, uint32 team, bool allowed_alt
}
// minfo is not actually used but the mount_id was updated
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(mount_id);
+ sObjectMgr->GetCreatureModelRandomGender(&mount_id);
return mount_id;
}
diff --git a/src/server/game/Globals/ObjectMgr.h b/src/server/game/Globals/ObjectMgr.h
index 4ed3a598c7e..736e1b0b60e 100755
--- a/src/server/game/Globals/ObjectMgr.h
+++ b/src/server/game/Globals/ObjectMgr.h
@@ -636,7 +636,7 @@ class ObjectMgr
CreatureTemplate const* GetCreatureTemplate(uint32 entry);
CreatureTemplateContainer const* GetCreatureTemplates() { return &CreatureTemplateStore; }
CreatureModelInfo const* GetCreatureModelInfo(uint32 modelId);
- CreatureModelInfo const* GetCreatureModelRandomGender(uint32 &displayID);
+ CreatureModelInfo const* GetCreatureModelRandomGender(uint32* displayID);
uint32 ChooseDisplayId(uint32 team, const CreatureTemplate *cinfo, const CreatureData *data = NULL);
static void ChooseCreatureFlags(const CreatureTemplate *cinfo, uint32& npcflag, uint32& unit_flags, uint32& dynamicflags, const CreatureData *data = NULL);
EquipmentInfo const *GetEquipmentInfo(uint32 entry);
diff --git a/src/server/game/Spells/Auras/SpellAuraEffects.cpp b/src/server/game/Spells/Auras/SpellAuraEffects.cpp
index 1246815aaaa..df674507f9a 100755
--- a/src/server/game/Spells/Auras/SpellAuraEffects.cpp
+++ b/src/server/game/Spells/Auras/SpellAuraEffects.cpp
@@ -3417,8 +3417,7 @@ void AuraEffect::HandleAuraTransform(AuraApplication const * aurApp, uint8 mode,
team = target->ToPlayer()->GetTeam();
uint32 displayID = sObjectMgr->ChooseDisplayId(team, ci);
-
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ sObjectMgr->GetCreatureModelRandomGender(&displayID);
target->SetUInt32Value(UNIT_FIELD_MOUNTDISPLAYID, displayID);
}
@@ -3878,7 +3877,7 @@ void AuraEffect::HandleAuraMounted(AuraApplication const * aurApp, uint8 mode, b
team = target->ToPlayer()->GetTeam();
uint32 displayID = sObjectMgr->ChooseDisplayId(team, ci);
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ sObjectMgr->GetCreatureModelRandomGender(&displayID);
//some spell has one aura of mount and one of vehicle
for (uint32 i = 0; i < MAX_SPELL_EFFECTS; ++i)
@@ -6154,7 +6153,7 @@ void AuraEffect::HandleAuraDummy(AuraApplication const * aurApp, uint8 mode, boo
team = target->ToPlayer()->GetTeam();
uint32 displayID = sObjectMgr->ChooseDisplayId(team, creatureInfo);
- CreatureModelInfo const *minfo = sObjectMgr->GetCreatureModelRandomGender(displayID);
+ sObjectMgr->GetCreatureModelRandomGender(&displayID);
target->SetUInt32Value(UNIT_FIELD_MOUNTDISPLAYID, displayID);
}
diff --git a/src/server/game/Tickets/TicketMgr.cpp b/src/server/game/Tickets/TicketMgr.cpp
index 7b1d0882ee9..cc124c28a88 100755
--- a/src/server/game/Tickets/TicketMgr.cpp
+++ b/src/server/game/Tickets/TicketMgr.cpp
@@ -100,7 +100,7 @@ void GmTicket::SaveToDB(SQLTransaction& trans) const
CharacterDatabase.ExecuteOrAppend(trans, stmt);
}
-void GmTicket::DeleteFromDB(SQLTransaction& trans)
+void GmTicket::DeleteFromDB()
{
PreparedStatement* stmt = CharacterDatabase.GetPreparedStatement(CHAR_DEL_GM_TICKET);
stmt->setUInt32(0, _id);
@@ -286,8 +286,7 @@ void TicketMgr::RemoveTicket(uint32 ticketId)
{
if (GmTicket* ticket = GetTicket(ticketId))
{
- SQLTransaction trans = SQLTransaction(NULL);
- ticket->DeleteFromDB(trans);
+ ticket->DeleteFromDB();
_ticketList.erase(ticketId);
}
}
diff --git a/src/server/game/Tickets/TicketMgr.h b/src/server/game/Tickets/TicketMgr.h
index c2e8b8d620b..2a5c99fcdb5 100755
--- a/src/server/game/Tickets/TicketMgr.h
+++ b/src/server/game/Tickets/TicketMgr.h
@@ -129,7 +129,7 @@ public:
bool LoadFromDB(Field* fields);
void SaveToDB(SQLTransaction& trans) const;
- void DeleteFromDB(SQLTransaction& trans);
+ void DeleteFromDB();
void WritePacket(WorldPacket& data) const;
void SendResponse(WorldSession* session) const;
diff --git a/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/culling_of_stratholme.cpp b/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/culling_of_stratholme.cpp
index 91028e8745a..7a1e8179151 100644
--- a/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/culling_of_stratholme.cpp
+++ b/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/culling_of_stratholme.cpp
@@ -1215,7 +1215,7 @@ class npc_crate_helper : public CreatureScript
_marked = false;
}
- void SpellHit(Unit* caster, SpellEntry const* spell)
+ void SpellHit(Unit* /*caster*/, SpellEntry const* spell)
{
if (spell->Id == SPELL_ARCANE_DISRUPTION && !_marked)
{
diff --git a/src/server/scripts/Kalimdor/bloodmyst_isle.cpp b/src/server/scripts/Kalimdor/bloodmyst_isle.cpp
index 632dd0925ac..4b00da28629 100644
--- a/src/server/scripts/Kalimdor/bloodmyst_isle.cpp
+++ b/src/server/scripts/Kalimdor/bloodmyst_isle.cpp
@@ -173,8 +173,8 @@ public:
{
go->SetGoState(GO_STATE_ACTIVE);
stillpine->GetMotionMaster()->MovePoint(1, go->GetPositionX(), go->GetPositionY()-15, go->GetPositionZ());
- player->CastedCreatureOrGO(NPC_PRINCESS_STILLPINE, 0, SPELL_OPENING_PRINCESS_STILLPINE_CREDIT);
- }
+ player->CastedCreatureOrGO(NPC_PRINCESS_STILLPINE, 0, SPELL_OPENING_PRINCESS_STILLPINE_CREDIT);
+ }
return true;
}
};
@@ -190,11 +190,10 @@ public:
void MovementInform(uint32 type, uint32 id)
{
- if (id == 1)
+ if (type == POINT_MOTION_TYPE && id == 1)
{
- DoScriptText(SAY_DIRECTION, me);
+ DoScriptText(SAY_DIRECTION, me);
me->ForcedDespawn();
- return;
}
}
};
diff --git a/src/server/scripts/Northrend/Nexus/EyeOfEternity/boss_malygos.cpp b/src/server/scripts/Northrend/Nexus/EyeOfEternity/boss_malygos.cpp
index 3111361a46e..e4640b766f6 100644
--- a/src/server/scripts/Northrend/Nexus/EyeOfEternity/boss_malygos.cpp
+++ b/src/server/scripts/Northrend/Nexus/EyeOfEternity/boss_malygos.cpp
@@ -74,7 +74,7 @@ enum Spells
SPELL_ARCANE_BREATH = 56272,
SPELL_ARCANE_STORM = 57459,
SPELL_BERSEKER = 60670,
-
+
SPELL_VORTEX_1 = 56237, // seems that frezze object animation
SPELL_VORTEX_2 = 55873, // visual effect
SPELL_VORTEX_3 = 56105, // this spell must handle all the script - casted by the boss and to himself
@@ -292,7 +292,7 @@ public:
{
if (!instance)
return;
-
+
SetPhase(PHASE_THREE, true);
// this despawns Hover Disks
@@ -313,7 +313,7 @@ public:
}
if (GameObject* go = GameObject::GetGameObject(*me, instance->GetData64(DATA_PLATFORM)))
- go->SetFlag(GAMEOBJECT_FLAGS, GO_FLAG_DESTROYED); // In sniffs it has this flag, but i don't know how is applied.
+ go->SetFlag(GAMEOBJECT_FLAGS, GO_FLAG_DESTROYED); // In sniffs it has this flag, but i don't know how is applied.
// pos sniffed
me->GetMotionMaster()->MoveIdle();
@@ -340,8 +340,8 @@ public:
events.ScheduleEvent(EVENT_YELL_2, 0, 0, _phase);
events.ScheduleEvent(EVENT_YELL_3, 8*IN_MILLISECONDS, 0, _phase);
events.ScheduleEvent(EVENT_YELL_4, 16*IN_MILLISECONDS, 0, _phase);
- events.ScheduleEvent(EVENT_SURGE_POWER_PHASE_3, (7, 16)*IN_MILLISECONDS, 0, _phase);
- events.ScheduleEvent(EVENT_STATIC_FIELD, (20, 30)*IN_MILLISECONDS, 0, _phase);
+ events.ScheduleEvent(EVENT_SURGE_POWER_PHASE_3, urand(7, 16)*IN_MILLISECONDS, 0, _phase);
+ events.ScheduleEvent(EVENT_STATIC_FIELD, urand(20, 30)*IN_MILLISECONDS, 0, _phase);
break;
default:
break;
@@ -400,7 +400,7 @@ public:
// not sure about the distance | I think it is better check this here than in the UpdateAI function...
if (who->GetDistance(me) <= 2.5f)
who->CastSpell(me, SPELL_POWER_SPARK_MALYGOS, true);
-
+
}
}
@@ -457,7 +457,7 @@ public:
me->GetMotionMaster()->MoveIdle();
me->GetMotionMaster()->MovePoint(MOVE_DEEP_BREATH_ROTATION, MalygosPhaseTwoWaypoints[0]);
-
+
Creature* summon = me->SummonCreature(NPC_HOVER_DISK_CASTER, HoverDiskWaypoints[MAX_HOVER_DISK_WAYPOINTS-1]);
if (summon && summon->IsAIEnabled)
summon->AI()->DoAction(ACTION_HOVER_DISK_START_WP_2);
@@ -474,7 +474,7 @@ public:
}
}
- void UpdateAI(const uint32 diff)
+ void UpdateAI(uint32 const diff)
{
if (!UpdateVictim())
return;
@@ -571,11 +571,11 @@ public:
break;
case EVENT_SURGE_POWER_PHASE_3:
DoCast(GetTargetPhaseThree(), SPELL_SURGE_POWER_PHASE_3);
- events.ScheduleEvent(EVENT_SURGE_POWER_PHASE_3, (7, 16)*IN_MILLISECONDS, 0, PHASE_THREE);
+ events.ScheduleEvent(EVENT_SURGE_POWER_PHASE_3, urand(7, 16)*IN_MILLISECONDS, 0, PHASE_THREE);
break;
case EVENT_STATIC_FIELD:
DoCast(GetTargetPhaseThree(), SPELL_STATIC_FIELD);
- events.ScheduleEvent(EVENT_STATIC_FIELD, (20, 30)*IN_MILLISECONDS, 0, PHASE_THREE);
+ events.ScheduleEvent(EVENT_STATIC_FIELD, urand(20, 30)*IN_MILLISECONDS, 0, PHASE_THREE);
break;
default:
break;
@@ -588,7 +588,7 @@ public:
Unit* GetTargetPhaseThree()
{
Unit* target = SelectTarget(SELECT_TARGET_RANDOM, 0);
-
+
// we are a drake
if (target->GetVehicleKit())
return target;
@@ -599,8 +599,8 @@ public:
if (Unit* base = target->GetVehicleBase())
return base;
}
-
- // is a player falling from a vehicle?
+
+ // is a player falling from a vehicle?
return NULL;
}
@@ -632,7 +632,7 @@ public:
class spell_malygos_vortex_dummy_SpellScript : public SpellScript
{
PrepareSpellScript(spell_malygos_vortex_dummy_SpellScript)
-
+
void HandleScript(SpellEffIndex /*effIndex*/)
{
Unit* caster = GetCaster();
@@ -730,7 +730,7 @@ public:
{
npc_portal_eoeAI(Creature* creature) : ScriptedAI(creature)
{
- instance = creature->GetInstanceScript();
+ instance = creature->GetInstanceScript();
}
void Reset()
@@ -738,7 +738,7 @@ public:
summonTimer = urand(5, 7)*IN_MILLISECONDS;
}
- void UpdateAI(const uint32 diff)
+ void UpdateAI(uint32 const diff)
{
if (!me->HasAura(SPELL_PORTAL_VISUAL_CLOSED) &&
!me->HasAura(SPELL_PORTAL_OPENED))
@@ -770,7 +770,7 @@ public:
{
summon->SetInCombatWithZone();
}
-
+
private:
uint32 summonTimer;
InstanceScript* instance;
@@ -794,7 +794,7 @@ public:
{
instance = creature->GetInstanceScript();
- MoveToMalygos();
+ MoveToMalygos();
}
void EnterEvadeMode()
@@ -812,8 +812,8 @@ public:
me->GetMotionMaster()->MoveFollow(malygos, 0.0f, 0.0f);
}
}
-
- void UpdateAI(const uint32 diff)
+
+ void UpdateAI(uint32 const /*diff*/)
{
if (!instance)
return;
@@ -864,8 +864,8 @@ public:
struct npc_hover_diskAI : public npc_escortAI
{
- npc_hover_diskAI(Creature* creature) : npc_escortAI(creature)
- {
+ npc_hover_diskAI(Creature* creature) : npc_escortAI(creature)
+ {
if (me->GetEntry() == NPC_HOVER_DISK_CASTER)
me->SetReactState(REACT_PASSIVE);
else
@@ -874,7 +874,7 @@ public:
instance = creature->GetInstanceScript();
}
- void PassengerBoarded(Unit* unit, int8 seat, bool apply)
+ void PassengerBoarded(Unit* unit, int8 /*seat*/, bool apply)
{
if (apply)
{
@@ -919,7 +919,7 @@ public:
// we dont evade
}
- void DoAction(const int32 action)
+ void DoAction(int32 const action)
{
if (me->GetEntry() != NPC_HOVER_DISK_CASTER)
return;
@@ -952,9 +952,8 @@ public:
// we dont do melee damage!
}
- void WaypointReached(uint32 i)
+ void WaypointReached(uint32 /*i*/)
{
-
}
private:
@@ -988,7 +987,7 @@ public:
DoCast(me, SPELL_ARCANE_OVERLOAD, false);
}
- void UpdateAI(const uint32 diff)
+ void UpdateAI(uint32 const /*diff*/)
{
// we dont do melee damage!
}
@@ -1087,7 +1086,7 @@ public:
events.ScheduleEvent(EVENT_YELL_1, 0);
}
- void UpdateAI(const uint32 diff)
+ void UpdateAI(uint32 const /*diff*/)
{
while (uint32 eventId = events.ExecuteEvent())
{
diff --git a/src/server/scripts/Northrend/Ulduar/ulduar/boss_razorscale.cpp b/src/server/scripts/Northrend/Ulduar/ulduar/boss_razorscale.cpp
index d3e31eebbf1..8b3f700c67d 100644
--- a/src/server/scripts/Northrend/Ulduar/ulduar/boss_razorscale.cpp
+++ b/src/server/scripts/Northrend/Ulduar/ulduar/boss_razorscale.cpp
@@ -286,7 +286,7 @@ class go_razorscale_harpoon : public GameObjectScript
public:
go_razorscale_harpoon() : GameObjectScript("go_razorscale_harpoon") {}
- bool OnGossipHello(Player* player, GameObject* go)
+ bool OnGossipHello(Player* /*player*/, GameObject* go)
{
InstanceScript* instance = go->GetInstanceScript();
if (Creature* razorscale = ObjectAccessor::GetCreature(*go, instance ? instance->GetData64(TYPE_RAZORSCALE) : 0))
@@ -360,7 +360,7 @@ class boss_razorscale : public CreatureScript
void MovementInform(uint32 type, uint32 id)
{
- if (id == 1)
+ if (type == POINT_MOTION_TYPE && id == 1)
{
phase = PHASE_GROUND;
events.SetPhase(PHASE_GROUND);
diff --git a/src/server/scripts/Outland/nagrand.cpp b/src/server/scripts/Outland/nagrand.cpp
index da8b2b491b5..5df55d19d3d 100644
--- a/src/server/scripts/Outland/nagrand.cpp
+++ b/src/server/scripts/Outland/nagrand.cpp
@@ -893,7 +893,7 @@ public:
};
};
-enum corki
+enum CorkiData
{
// first quest
QUEST_HELP = 9923,
@@ -990,10 +990,10 @@ public:
else
Say_Timer -= diff;
}
-
+
void MovementInform(uint32 type, uint32 id)
{
- if (id == 1)
+ if (type == POINT_MOTION_TYPE && id == 1)
{
Say_Timer = 5000;
ReleasedFromCage = true;