[XTarget] Performance Improvement After Corpse Change (#1918)

Removed a conditional that was rendered obsolete by moving the addition of a
mob to the auto haters list to fire after an IsValidXTarget check.  This
made an entity_list call unnecessary.  [zone/client.cpp]

Removed said unnecessary entity_list call.  [zone/client.cpp]

Removed a superfluous call to ProcessXTargetAutoHaters [zone/attack.cpp]
This commit is contained in:
mmcgarvey 2022-01-12 17:04:18 -05:00 committed by GitHub
parent d10145fc6f
commit 1a556f4451
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 10 deletions

View File

@ -1799,7 +1799,6 @@ bool Client::Death(Mob* killerMob, int32 damage, uint16 spell, EQ::skills::Skill
entity_list.RemoveFromTargets(this, true);
hate_list.RemoveEntFromHateList(this);
RemoveAutoXTargets();
ProcessXTargetAutoHaters();
//remove ourself from all proximities
ClearAllProximities();

View File

@ -7426,9 +7426,7 @@ void Client::ProcessXTargetAutoHaters()
std::queue<int> empty_slots;
for (int i = 0; i < GetMaxXTargets(); ++i) {
if (XTargets[i].Type != Auto)
continue;
auto *mob = entity_list.GetMob(XTargets[i].ID);
continue;
if (XTargets[i].ID != 0 && !GetXTargetAutoMgr()->contains_mob(XTargets[i].ID)) {
XTargets[i].ID = 0;
@ -7436,12 +7434,6 @@ void Client::ProcessXTargetAutoHaters()
XTargets[i].dirty = true;
}
if (XTargets[i].ID != 0 && mob && !mob->IsValidXTarget()) {
XTargets[i].ID = 0;
XTargets[i].Name[0] = 0;
XTargets[i].dirty = true;
}
if (XTargets[i].ID == 0) {
empty_slots.push(i);
continue;