[Cleanup] Possible issues with variable/parameter name equality. (#2161)

This commit is contained in:
Kinglykrab 2022-05-09 21:00:37 -04:00 committed by GitHub
parent a1a4f91ea6
commit 6e0d101457
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 27 additions and 30 deletions

View File

@ -28,20 +28,17 @@ Adventure::Adventure(AdventureTemplate *t)
instance_id = 0; instance_id = 0;
} }
Adventure::Adventure(AdventureTemplate *t, int count, int assassination_count, AdventureStatus status, uint16 instance_id, uint32 time_left) Adventure::Adventure(AdventureTemplate *t, int in_count, int in_assassination_count, AdventureStatus in_status, uint16 in_instance_id, uint32 time_left)
{ {
adventure_template = t; adventure_template = t;
count = count; count = in_count;
assassination_count = assassination_count; assassination_count = in_assassination_count;
status = status; status = in_status;
instance_id = instance_id; instance_id = in_instance_id;
if(status == AS_Finished) if (in_status == AS_Finished) {
{
database.SetInstanceDuration(instance_id, time_left); database.SetInstanceDuration(instance_id, time_left);
} } else {
else
{
database.SetInstanceDuration(instance_id, time_left + 60); database.SetInstanceDuration(instance_id, time_left + 60);
} }

View File

@ -65,7 +65,7 @@ class Adventure
{ {
public: public:
Adventure(AdventureTemplate *t); Adventure(AdventureTemplate *t);
Adventure(AdventureTemplate *t, int count, int assassination_count, AdventureStatus status, uint16 instance_id, uint32 time_left); Adventure(AdventureTemplate *t, int in_count, int in_assassination_count, AdventureStatus in_status, uint16 in_instance_id, uint32 time_left);
~Adventure(); ~Adventure();
bool Process(); bool Process();
bool IsActive(); bool IsActive();

View File

@ -93,23 +93,23 @@ ZoneServer::~ZoneServer() {
} }
} }
bool ZoneServer::SetZone(uint32 zone_id, uint32 instance_id, bool is_static_zone) { bool ZoneServer::SetZone(uint32 in_zone_id, uint32 in_instance_id, bool is_static_zone) {
is_booting_up = false; is_booting_up = false;
std::string zone_short_name = ZoneName(zone_id, true); std::string zone_short_name = ZoneName(in_zone_id, true);
std::string zone_long_name = ZoneLongName(zone_id, true); std::string zone_long_name = ZoneLongName(in_zone_id, true);
if (zone_id) { if (in_zone_id) {
LogInfo( LogInfo(
"Setting zone process to Zone: {} ({}) ID: {}{}{}", "Setting zone process to Zone: {} ({}) ID: {}{}{}",
zone_long_name, zone_long_name,
zone_short_name, zone_short_name,
zone_id, in_zone_id,
( (
instance_id ? in_instance_id ?
fmt::format( fmt::format(
" (Instance ID {})", " (Instance ID {})",
instance_id in_instance_id
) : ) :
"" ""
), ),
@ -117,10 +117,10 @@ bool ZoneServer::SetZone(uint32 zone_id, uint32 instance_id, bool is_static_zone
); );
} }
zone_server_zone_id = zone_id; zone_server_zone_id = in_zone_id;
instance_id = instance_id; instance_id = in_instance_id;
if (zone_id) { if (in_zone_id) {
zone_server_previous_zone_id = zone_id; zone_server_previous_zone_id = in_zone_id;
} }
if (!zone_server_zone_id) { if (!zone_server_zone_id) {
@ -1504,10 +1504,10 @@ void ZoneServer::ChangeWID(uint32 iCharID, uint32 iWID) {
} }
void ZoneServer::TriggerBootup(uint32 zone_id, uint32 instance_id, const char* admin_name, bool is_static_zone) { void ZoneServer::TriggerBootup(uint32 in_zone_id, uint32 in_instance_id, const char* admin_name, bool is_static_zone) {
is_booting_up = true; is_booting_up = true;
zone_server_zone_id = zone_id; zone_server_zone_id = in_zone_id;
instance_id = instance_id; instance_id = in_instance_id;
auto pack = new ServerPacket(ServerOP_ZoneBootup, sizeof(ServerZoneStateChange_struct)); auto pack = new ServerPacket(ServerOP_ZoneBootup, sizeof(ServerZoneStateChange_struct));
auto s = (ServerZoneStateChange_struct*) pack->pBuffer; auto s = (ServerZoneStateChange_struct*) pack->pBuffer;
@ -1516,12 +1516,12 @@ void ZoneServer::TriggerBootup(uint32 zone_id, uint32 instance_id, const char* a
strn0cpy(s->adminname, admin_name, sizeof(s->adminname)); strn0cpy(s->adminname, admin_name, sizeof(s->adminname));
} }
s->zoneid = zone_id ? zone_id : GetZoneID(); s->zoneid = in_zone_id ? in_zone_id : GetZoneID();
s->instanceid = instance_id; s->instanceid = in_instance_id;
s->makestatic = is_static_zone; s->makestatic = is_static_zone;
SendPacket(pack); SendPacket(pack);
delete pack; delete pack;
LSBootUpdate(zone_id, instance_id); LSBootUpdate(in_zone_id, in_instance_id);
} }
void ZoneServer::IncomingClient(Client* client) { void ZoneServer::IncomingClient(Client* client) {

View File

@ -40,8 +40,8 @@ public:
void SendEmoteMessage(const char* to, uint32 to_guilddbid, int16 to_minstatus, uint32 type, const char* message, ...); void SendEmoteMessage(const char* to, uint32 to_guilddbid, int16 to_minstatus, uint32 type, const char* message, ...);
void SendEmoteMessageRaw(const char* to, uint32 to_guilddbid, int16 to_minstatus, uint32 type, const char* message); void SendEmoteMessageRaw(const char* to, uint32 to_guilddbid, int16 to_minstatus, uint32 type, const char* message);
void SendKeepAlive(); void SendKeepAlive();
bool SetZone(uint32 zone_id, uint32 instance_id = 0, bool is_static_zone = false); bool SetZone(uint32 in_zone_id, uint32 in_instance_id = 0, bool is_static_zone = false);
void TriggerBootup(uint32 zone_id = 0, uint32 instance_id = 0, const char* admin_name = 0, bool is_static_zone = false); void TriggerBootup(uint32 in_zone_id = 0, uint32 in_instance_id = 0, const char* admin_name = 0, bool is_static_zone = false);
void Disconnect() { auto handle = tcpc->Handle(); if (handle) { handle->Disconnect(); } } void Disconnect() { auto handle = tcpc->Handle(); if (handle) { handle->Disconnect(); } }
void IncomingClient(Client* client); void IncomingClient(Client* client);
void LSBootUpdate(uint32 zone_id, uint32 instance_id = 0, bool startup = false); void LSBootUpdate(uint32 zone_id, uint32 instance_id = 0, bool startup = false);