cls cleanup

This commit is contained in:
nytmyr
2024-12-20 00:44:49 -06:00
parent 21f8bc0f95
commit eaa1f67802
4 changed files with 17 additions and 18 deletions
+10 -10
View File
@@ -2791,7 +2791,7 @@ bool IsLichSpell(uint16 spell_id)
return false; return false;
} }
bool IsBotSpellTypeDetrimental(uint16 spellType, uint8 cls) { bool IsBotSpellTypeDetrimental(uint16 spellType) {
switch (spellType) { switch (spellType) {
case BotSpellTypes::Nuke: case BotSpellTypes::Nuke:
case BotSpellTypes::Root: case BotSpellTypes::Root:
@@ -2831,7 +2831,7 @@ bool IsBotSpellTypeDetrimental(uint16 spellType, uint8 cls) {
return false; return false;
} }
bool IsBotSpellTypeBeneficial(uint16 spellType, uint8 cls) { bool IsBotSpellTypeBeneficial(uint16 spellType) {
switch (spellType) { switch (spellType) {
case BotSpellTypes::RegularHeal: case BotSpellTypes::RegularHeal:
case BotSpellTypes::CompleteHeal: case BotSpellTypes::CompleteHeal:
@@ -2999,12 +2999,12 @@ bool IsGroupBotSpellType(uint16 spellType) {
bool IsGroupTargetOnlyBotSpellType(uint16 spellType) { bool IsGroupTargetOnlyBotSpellType(uint16 spellType) {
switch (spellType) { switch (spellType) {
case BotSpellTypes::GroupCures: case BotSpellTypes::GroupCures:
case BotSpellTypes::GroupCompleteHeals: case BotSpellTypes::GroupCompleteHeals:
case BotSpellTypes::GroupHeals: case BotSpellTypes::GroupHeals:
return true; return true;
default: default:
return false; return false;
} }
return false; return false;
@@ -3082,7 +3082,7 @@ bool IsHealBotSpellType(uint16 spellType) {
return false; return false;
} }
bool SpellTypeRequiresLoS(uint16 spellType, uint16 cls) { bool SpellTypeRequiresLoS(uint16 spellType) {
if (IsAEBotSpellType(spellType)) { // These gather their own targets later if (IsAEBotSpellType(spellType)) { // These gather their own targets later
return false; return false;
} }
@@ -3110,7 +3110,7 @@ bool SpellTypeRequiresLoS(uint16 spellType, uint16 cls) {
return true; return true;
} }
bool SpellTypeRequiresTarget(uint16 spellType, uint16 cls) { bool SpellTypeRequiresTarget(uint16 spellType) {
switch (spellType) { switch (spellType) {
case BotSpellTypes::Pet: case BotSpellTypes::Pet:
case BotSpellTypes::Succor: case BotSpellTypes::Succor:
+4 -4
View File
@@ -737,8 +737,8 @@ const uint32 SPELL_TYPES_DETRIMENTAL = (SpellType_Nuke | SpellType_Root | SpellT
const uint32 SPELL_TYPES_BENEFICIAL = (SpellType_Heal | SpellType_Buff | SpellType_Escape | SpellType_Pet | SpellType_InCombatBuff | SpellType_Cure | SpellType_HateRedux | SpellType_InCombatBuffSong | SpellType_OutOfCombatBuffSong | SpellType_PreCombatBuff | SpellType_PreCombatBuffSong); const uint32 SPELL_TYPES_BENEFICIAL = (SpellType_Heal | SpellType_Buff | SpellType_Escape | SpellType_Pet | SpellType_InCombatBuff | SpellType_Cure | SpellType_HateRedux | SpellType_InCombatBuffSong | SpellType_OutOfCombatBuffSong | SpellType_PreCombatBuff | SpellType_PreCombatBuffSong);
const uint32 SPELL_TYPES_INNATE = (SpellType_Nuke | SpellType_Lifetap | SpellType_DOT | SpellType_Dispel | SpellType_Mez | SpellType_Slow | SpellType_Debuff | SpellType_Charm | SpellType_Root); const uint32 SPELL_TYPES_INNATE = (SpellType_Nuke | SpellType_Lifetap | SpellType_DOT | SpellType_Dispel | SpellType_Mez | SpellType_Slow | SpellType_Debuff | SpellType_Charm | SpellType_Root);
bool IsBotSpellTypeDetrimental (uint16 spellType, uint8 cls = 0); bool IsBotSpellTypeDetrimental (uint16 spellType);
bool IsBotSpellTypeBeneficial (uint16 spellType, uint8 cls = 0); bool IsBotSpellTypeBeneficial (uint16 spellType);
bool IsBotSpellTypeOtherBeneficial(uint16 spellType); bool IsBotSpellTypeOtherBeneficial(uint16 spellType);
bool IsBotSpellTypeInnate (uint16 spellType); bool IsBotSpellTypeInnate (uint16 spellType);
bool IsAEBotSpellType(uint16 spellType); bool IsAEBotSpellType(uint16 spellType);
@@ -747,8 +747,8 @@ bool IsGroupTargetOnlyBotSpellType(uint16 spellType);
bool IsPetBotSpellType(uint16 spellType); bool IsPetBotSpellType(uint16 spellType);
bool IsClientBotSpellType(uint16 spellType); bool IsClientBotSpellType(uint16 spellType);
bool IsHealBotSpellType(uint16 spellType); bool IsHealBotSpellType(uint16 spellType);
bool SpellTypeRequiresLoS(uint16 spellType, uint16 cls = 0); bool SpellTypeRequiresLoS(uint16 spellType);
bool SpellTypeRequiresTarget(uint16 spellType, uint16 cls = 0); bool SpellTypeRequiresTarget(uint16 spellType);
bool SpellTypeRequiresAEChecks(uint16 spellType); bool SpellTypeRequiresAEChecks(uint16 spellType);
bool IsCommandedSpellType(uint16 spellType); bool IsCommandedSpellType(uint16 spellType);
bool IsPullingSpellType(uint16 spellType); bool IsPullingSpellType(uint16 spellType);
+2 -3
View File
@@ -9661,7 +9661,6 @@ bool Bot::CanCastSpellType(uint16 spellType, uint16 spell_id, Mob* tar) {
} }
uint8 botClass = GetClass(); uint8 botClass = GetClass();
//uint8 botLevel = GetLevel();
switch (spellType) { switch (spellType) {
case BotSpellTypes::Buff: case BotSpellTypes::Buff:
@@ -10484,7 +10483,7 @@ uint16 Bot::GetDefaultSpellTypePriority(uint16 spellType, uint8 priorityType, ui
} }
uint16 Bot::GetDefaultSpellTypeIdlePriority(uint16 spellType, uint8 botClass, uint8 stance) { uint16 Bot::GetDefaultSpellTypeIdlePriority(uint16 spellType, uint8 botClass, uint8 stance) {
if (!IsBotSpellTypeBeneficial(spellType, botClass)) { if (!IsBotSpellTypeBeneficial(spellType)) {
return 0; return 0;
} }
@@ -10746,7 +10745,7 @@ uint16 Bot::GetDefaultSpellTypePursuePriority(uint16 spellType, uint8 botClass,
uint16 Bot::GetDefaultSpellTypeResistLimit(uint16 spellType, uint8 stance) { uint16 Bot::GetDefaultSpellTypeResistLimit(uint16 spellType, uint8 stance) {
if (!IsBotSpellTypeBeneficial(spellType, GetClass())) { if (!IsBotSpellTypeBeneficial(spellType)) {
return RuleI(Bots, SpellResistLimit); return RuleI(Bots, SpellResistLimit);
} }
else { else {
+1 -1
View File
@@ -63,7 +63,7 @@ bool Bot::AICastSpell(Mob* tar, uint8 iChance, uint16 spellType, uint16 subTarge
botSpell.SpellIndex = 0; botSpell.SpellIndex = 0;
botSpell.ManaCost = 0; botSpell.ManaCost = 0;
if (SpellTypeRequiresLoS(spellType, botClass) && tar != this) { if (SpellTypeRequiresLoS(spellType) && tar != this) {
SetHasLoS(DoLosChecks(this, tar)); SetHasLoS(DoLosChecks(this, tar));
} }
else { else {