mirror of
https://github.com/mangosfour/server.git
synced 2025-12-13 22:37:03 +00:00
Fixed some format arg type/value pairs. Other warnings.
This commit is contained in:
parent
b62f376d73
commit
004bdf1d3c
14 changed files with 30 additions and 20 deletions
|
|
@ -998,7 +998,7 @@ bool BattleGround::DelCreature(uint32 type)
|
||||||
Creature *cr = HashMapHolder<Creature>::Find(m_BgCreatures[type]);
|
Creature *cr = HashMapHolder<Creature>::Find(m_BgCreatures[type]);
|
||||||
if(!cr)
|
if(!cr)
|
||||||
{
|
{
|
||||||
sLog.outError("Can't find creature guid: %u",m_BgCreatures[type]);
|
sLog.outError("Can't find creature guid: %u",GUID_LOPART(m_BgCreatures[type]));
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
cr->CleanupsBeforeDelete();
|
cr->CleanupsBeforeDelete();
|
||||||
|
|
@ -1012,7 +1012,7 @@ bool BattleGround::DelObject(uint32 type)
|
||||||
GameObject *obj = HashMapHolder<GameObject>::Find(m_BgObjects[type]);
|
GameObject *obj = HashMapHolder<GameObject>::Find(m_BgObjects[type]);
|
||||||
if(!obj)
|
if(!obj)
|
||||||
{
|
{
|
||||||
sLog.outError("Can't find gobject guid: %u",m_BgObjects[type]);
|
sLog.outError("Can't find gobject guid: %u",GUID_LOPART(m_BgObjects[type]));
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
obj->SetRespawnTime(0); // not save respawn time
|
obj->SetRespawnTime(0); // not save respawn time
|
||||||
|
|
|
||||||
|
|
@ -104,7 +104,7 @@ void BattleGroundBE::AddPlayer(Player *plr)
|
||||||
m_PlayerScores[plr->GetGUID()] = sc;
|
m_PlayerScores[plr->GetGUID()] = sc;
|
||||||
}
|
}
|
||||||
|
|
||||||
void BattleGroundBE::RemovePlayer(Player *plr, uint64 guid)
|
void BattleGroundBE::RemovePlayer(Player* /*plr*/, uint64 /*guid*/)
|
||||||
{
|
{
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -556,7 +556,7 @@ void BattleGroundEY::RespawnFlagAfterDrop()
|
||||||
if(obj)
|
if(obj)
|
||||||
obj->Delete();
|
obj->Delete();
|
||||||
else
|
else
|
||||||
sLog.outError("BattleGroundEY: Unknown dropped flag guid: %u",GetDroppedFlagGUID());
|
sLog.outError("BattleGroundEY: Unknown dropped flag guid: %u",GUID_LOPART(GetDroppedFlagGUID()));
|
||||||
|
|
||||||
SetDroppedFlagGUID(0);
|
SetDroppedFlagGUID(0);
|
||||||
}
|
}
|
||||||
|
|
@ -741,7 +741,8 @@ void BattleGroundEY::EventTeamCapturedPoint(Player *Source, uint32 Point)
|
||||||
WorldSafeLocsEntry const *sg = NULL;
|
WorldSafeLocsEntry const *sg = NULL;
|
||||||
sg = sWorldSafeLocsStore.LookupEntry(m_CapturingPointTypes[Point].GraveYardId);
|
sg = sWorldSafeLocsStore.LookupEntry(m_CapturingPointTypes[Point].GraveYardId);
|
||||||
if(!sg || !AddSpiritGuide(Point, sg->x, sg->y, sg->z, 3.124139f, Team))
|
if(!sg || !AddSpiritGuide(Point, sg->x, sg->y, sg->z, 3.124139f, Team))
|
||||||
sLog.outError("BatteGroundEY: Failed to spawn spirit guide! point: %u, team: u, graveyard_id: %u", Point, Team, m_CapturingPointTypes[Point].GraveYardId);
|
sLog.outError("BatteGroundEY: Failed to spawn spirit guide! point: %u, team: %u, graveyard_id: %u",
|
||||||
|
Point, Team, m_CapturingPointTypes[Point].GraveYardId);
|
||||||
|
|
||||||
UpdatePointsIcons(Team, Point);
|
UpdatePointsIcons(Team, Point);
|
||||||
UpdatePointsCount(Team);
|
UpdatePointsCount(Team);
|
||||||
|
|
|
||||||
|
|
@ -341,7 +341,7 @@ void BattleGroundQueue::Update(uint32 bgTypeId, uint32 queue_id)
|
||||||
/*** BATTLEGROUND QUEUE EVENTS ***/
|
/*** BATTLEGROUND QUEUE EVENTS ***/
|
||||||
/*********************************************************/
|
/*********************************************************/
|
||||||
|
|
||||||
bool BGQueueInviteEvent::Execute(uint64 /*e_time*/, uint32 p_time)
|
bool BGQueueInviteEvent::Execute(uint64 /*e_time*/, uint32 /*p_time*/)
|
||||||
{
|
{
|
||||||
Player* plr = objmgr.GetPlayer( m_PlayerGuid );
|
Player* plr = objmgr.GetPlayer( m_PlayerGuid );
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -98,7 +98,7 @@ void BattleGroundRL::AddPlayer(Player *plr)
|
||||||
m_PlayerScores[plr->GetGUID()] = sc;
|
m_PlayerScores[plr->GetGUID()] = sc;
|
||||||
}
|
}
|
||||||
|
|
||||||
void BattleGroundRL::RemovePlayer(Player *plr, uint64 guid)
|
void BattleGroundRL::RemovePlayer(Player* /*plr*/, uint64 /*guid*/)
|
||||||
{
|
{
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -198,7 +198,7 @@ void BattleGroundWS::RespawnFlagAfterDrop(uint32 team)
|
||||||
if(obj)
|
if(obj)
|
||||||
obj->Delete();
|
obj->Delete();
|
||||||
else
|
else
|
||||||
sLog.outError("unknown droped flag bg, guid: %u",GetDroppedFlagGUID(team));
|
sLog.outError("unknown droped flag bg, guid: %u",GUID_LOPART(GetDroppedFlagGUID(team)));
|
||||||
|
|
||||||
SetDroppedFlagGUID(0,team);
|
SetDroppedFlagGUID(0,team);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -758,7 +758,8 @@ void WorldSession::HandlePlayerLogin(LoginQueryHolder * holder)
|
||||||
SendNotification(LANG_GM_ON);
|
SendNotification(LANG_GM_ON);
|
||||||
|
|
||||||
std::string IP_str = GetRemoteAddress();
|
std::string IP_str = GetRemoteAddress();
|
||||||
sLog.outChar("Account: %d (IP: %s) Login Character:[%s] (guid:%u)",GetAccountId(),IP_str.c_str(),pCurrChar->GetName() ,pCurrChar->GetGUID());
|
sLog.outChar("Account: %d (IP: %s) Login Character:[%s] (guid:%u)",
|
||||||
|
GetAccountId(),IP_str.c_str(),pCurrChar->GetName() ,pCurrChar->GetGUIDLow());
|
||||||
|
|
||||||
m_playerLoading = false;
|
m_playerLoading = false;
|
||||||
delete holder;
|
delete holder;
|
||||||
|
|
|
||||||
|
|
@ -79,7 +79,8 @@ bool Corpse::Create( uint32 guidlow, Player *owner, uint32 mapid, float x, float
|
||||||
|
|
||||||
if(!IsPositionValid())
|
if(!IsPositionValid())
|
||||||
{
|
{
|
||||||
sLog.outError("ERROR: Corpse (guidlow %d, owner %s) not created. Suggested coordinates isn't valid (X: %d Y: ^%d)",guidlow,owner->GetName(),x,y);
|
sLog.outError("ERROR: Corpse (guidlow %d, owner %s) not created. Suggested coordinates isn't valid (X: %f Y: %f)",
|
||||||
|
guidlow,owner->GetName(),x,y);
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -196,7 +197,8 @@ bool Corpse::LoadFromDB(uint32 guid, Field *fields)
|
||||||
|
|
||||||
if(!IsPositionValid())
|
if(!IsPositionValid())
|
||||||
{
|
{
|
||||||
sLog.outError("ERROR: Corpse (guidlow %d, owner %d) not created. Suggested coordinates isn't valid (X: %d Y: ^%d)",GetGUIDLow(),GUID_LOPART(GetOwnerGUID()),GetPositionX(),GetPositionY());
|
sLog.outError("ERROR: Corpse (guidlow %d, owner %d) not created. Suggested coordinates isn't valid (X: %f Y: %f)",
|
||||||
|
GetGUIDLow(),GUID_LOPART(GetOwnerGUID()),GetPositionX(),GetPositionY());
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -268,7 +268,8 @@ bool LootStoreItem::IsValid(LootStore const& store, uint32 entry) const
|
||||||
|
|
||||||
if( chance != 0 && chance < 0.000001f ) // loot with low chance
|
if( chance != 0 && chance < 0.000001f ) // loot with low chance
|
||||||
{
|
{
|
||||||
sLog.outErrorDb("Table '%s' entry %d item %d: low chance (%d) - skipped", store.GetName(), entry, itemid, chance);
|
sLog.outErrorDb("Table '%s' entry %d item %d: low chance (%f) - skipped",
|
||||||
|
store.GetName(), entry, itemid, chance);
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -133,7 +133,7 @@ void WorldSession::HandleWhoOpcode( WorldPacket & recv_data )
|
||||||
|
|
||||||
wstrToLower(str[i]);
|
wstrToLower(str[i]);
|
||||||
|
|
||||||
sLog.outDebug("String %u: %s", i, str[i].c_str());
|
sLog.outDebug("String %u: %s", i, temp.c_str());
|
||||||
}
|
}
|
||||||
|
|
||||||
std::wstring wplayer_name;
|
std::wstring wplayer_name;
|
||||||
|
|
@ -969,7 +969,7 @@ void WorldSession::HandleAreaTriggerOpcode(WorldPacket & recv_data)
|
||||||
(fabs(rotateDy) > atEntry->box_y/2 + delta) ||
|
(fabs(rotateDy) > atEntry->box_y/2 + delta) ||
|
||||||
(fabs(dz) > atEntry->box_z/2 + delta) )
|
(fabs(dz) > atEntry->box_z/2 + delta) )
|
||||||
{
|
{
|
||||||
sLog.outDebug("Player '%s' (GUID: %u) too far (1/2 box X: %f 1/2 box Y: %u 1/2 box Z: %u rotate dX: %f rotate dY: %f dZ:%f), ignore Area Trigger ID: %u",
|
sLog.outDebug("Player '%s' (GUID: %u) too far (1/2 box X: %f 1/2 box Y: %f 1/2 box Z: %f rotate dX: %f rotate dY: %f dZ:%f), ignore Area Trigger ID: %u",
|
||||||
pl->GetName(), pl->GetGUIDLow(), atEntry->box_x/2, atEntry->box_y/2, atEntry->box_z/2, rotateDx, rotateDy, dz, Trigger_ID);
|
pl->GetName(), pl->GetGUIDLow(), atEntry->box_x/2, atEntry->box_y/2, atEntry->box_z/2, rotateDx, rotateDy, dz, Trigger_ID);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -186,6 +186,7 @@ MotionMaster::MoveChase(Unit* target, float dist, float angle)
|
||||||
if(i_owner->GetTypeId()==TYPEID_PLAYER)
|
if(i_owner->GetTypeId()==TYPEID_PLAYER)
|
||||||
{
|
{
|
||||||
DEBUG_LOG("Player (GUID: %u) chase to %s (GUID: %u)",
|
DEBUG_LOG("Player (GUID: %u) chase to %s (GUID: %u)",
|
||||||
|
i_owner->GetGUIDLow(),
|
||||||
target->GetTypeId()==TYPEID_PLAYER ? "player" : "creature",
|
target->GetTypeId()==TYPEID_PLAYER ? "player" : "creature",
|
||||||
target->GetTypeId()==TYPEID_PLAYER ? i_owner->GetGUIDLow() : ((Creature*)i_owner)->GetDBTableGUIDLow() );
|
target->GetTypeId()==TYPEID_PLAYER ? i_owner->GetGUIDLow() : ((Creature*)i_owner)->GetDBTableGUIDLow() );
|
||||||
Mutate(new TargetedMovementGenerator<Player>(*target,dist,angle));
|
Mutate(new TargetedMovementGenerator<Player>(*target,dist,angle));
|
||||||
|
|
|
||||||
|
|
@ -137,14 +137,15 @@ void WorldSession::SendTrainerList( uint64 guid,std::string strTitle )
|
||||||
|
|
||||||
if (!ci)
|
if (!ci)
|
||||||
{
|
{
|
||||||
sLog.outDebug( "WORLD: SendTrainerList - (%u) NO CREATUREINFO! (GUID: %u)", uint32(GUID_LOPART(guid)), guid );
|
sLog.outDebug( "WORLD: SendTrainerList - (GUID: %u) NO CREATUREINFO!",GUID_LOPART(guid) );
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
TrainerSpellData const* trainer_spells = unit->GetTrainerSpells();
|
TrainerSpellData const* trainer_spells = unit->GetTrainerSpells();
|
||||||
if(!trainer_spells)
|
if(!trainer_spells)
|
||||||
{
|
{
|
||||||
sLog.outDebug( "WORLD: SendTrainerList - Training spells not found for creature (GUID: %u Entry: %u)", guid, unit->GetEntry());
|
sLog.outDebug( "WORLD: SendTrainerList - Training spells not found for creature (GUID: %u Entry: %u)",
|
||||||
|
GUID_LOPART(guid), unit->GetEntry());
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -161,7 +161,8 @@ bool Pet::LoadPetFromDB( Unit* owner, uint32 petentry, uint32 petnumber, bool cu
|
||||||
|
|
||||||
if(!IsPositionValid())
|
if(!IsPositionValid())
|
||||||
{
|
{
|
||||||
sLog.outError("ERROR: Pet (guidlow %d, entry %d) not loaded. Suggested coordinates isn't valid (X: %d Y: ^%d)", GetGUIDLow(), GetEntry(), GetPositionX(), GetPositionY());
|
sLog.outError("ERROR: Pet (guidlow %d, entry %d) not loaded. Suggested coordinates isn't valid (X: %f Y: %f)",
|
||||||
|
GetGUIDLow(), GetEntry(), GetPositionX(), GetPositionY());
|
||||||
delete result;
|
delete result;
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
@ -179,7 +180,7 @@ bool Pet::LoadPetFromDB( Unit* owner, uint32 petentry, uint32 petnumber, bool cu
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
if(getPetType() == HUNTER_PET || getPetType() == SUMMON_PET && cinfo->type == CREATURE_TYPE_DEMON && owner->getClass() == CLASS_WARLOCK)
|
if(getPetType() == HUNTER_PET || (getPetType() == SUMMON_PET && cinfo->type == CREATURE_TYPE_DEMON && owner->getClass() == CLASS_WARLOCK))
|
||||||
m_charmInfo->SetPetNumber(pet_number, true);
|
m_charmInfo->SetPetNumber(pet_number, true);
|
||||||
else
|
else
|
||||||
m_charmInfo->SetPetNumber(pet_number, false);
|
m_charmInfo->SetPetNumber(pet_number, false);
|
||||||
|
|
@ -734,7 +735,8 @@ bool Pet::CreateBaseAtCreature(Creature* creature)
|
||||||
|
|
||||||
if(!IsPositionValid())
|
if(!IsPositionValid())
|
||||||
{
|
{
|
||||||
sLog.outError("ERROR: Pet (guidlow %d, entry %d) not created base at creature. Suggested coordinates isn't valid (X: %d Y: ^%d)", GetGUIDLow(), GetEntry(), GetPositionX(), GetPositionY());
|
sLog.outError("ERROR: Pet (guidlow %d, entry %d) not created base at creature. Suggested coordinates isn't valid (X: %f Y: %f)",
|
||||||
|
GetGUIDLow(), GetEntry(), GetPositionX(), GetPositionY());
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -100,7 +100,8 @@ bool SqlQueryHolder::SetQuery(size_t index, const char *sql)
|
||||||
|
|
||||||
if(m_queries[index].first != NULL)
|
if(m_queries[index].first != NULL)
|
||||||
{
|
{
|
||||||
sLog.outError("Attempt assign query to holder index (%u) where other query stored (Old: [%s] New: [%s])",index,m_queries.size(),m_queries[index].first,sql);
|
sLog.outError("Attempt assign query to holder index (%u) where other query stored (Old: [%s] New: [%s])",
|
||||||
|
index,m_queries[index].first,sql);
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue