Core: SpellInRange: For Rogues and Death Knight consider Melee Range …

…as well.
Xian55 · May 29, 2024 · 3e57334 · 3e57334
1 parent 38d1018
commit 3e57334
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Core/AddonComponent/SpellInRange.cs b/Core/AddonComponent/SpellInRange.cs
@@ -132,15 +132,15 @@ public void Update(IAddonDataProvider reader)
     public bool WithinCombatRange(PlayerReader playerReader, UnitClass @class) => @class switch
     {
         UnitClass.Warrior => (playerReader.Level.Value >= 4 && Warrior_Rend) || playerReader.IsInMeleeRange(),
-        UnitClass.Rogue => Rogue_SinisterStrike,
+        UnitClass.Rogue => Rogue_SinisterStrike || playerReader.IsInMeleeRange(),
         UnitClass.Priest => Priest_Smite,
         UnitClass.Druid => Druid_Wrath || playerReader.IsInMeleeRange(),
         UnitClass.Paladin => (playerReader.Level.Value >= 4 && Paladin_Judgement) || playerReader.IsInMeleeRange(),
         UnitClass.Mage => Mage_Frostbolt || Mage_Fireball,
         UnitClass.Hunter => (playerReader.Level.Value >= 4 && Hunter_SerpentSting) || Hunter_AutoShoot || playerReader.IsInMeleeRange(),
         UnitClass.Warlock => Warlock_ShadowBolt,
         UnitClass.Shaman => Shaman_LightningBolt,
-        UnitClass.DeathKnight => DeathKnight_IcyTouch,
+        UnitClass.DeathKnight => DeathKnight_IcyTouch || playerReader.IsInMeleeRange(),
         _ => true
     };
 }
diff --git a/Game/WoWProcess/WowProcess.cs b/Game/WoWProcess/WowProcess.cs
@@ -10,13 +10,13 @@ namespace Game;
 
 public sealed class WowProcess
 {
-    private static readonly string[] defaultProcessNames = new string[] {
+    private static readonly string[] defaultProcessNames = [
         "Wow",
         "WowClassic",
         "WowClassicT",
         "Wow-64",
         "WowClassicB"
-    };
+    ];
 
     private readonly Thread thread;
     private readonly CancellationToken token;