Pro Warrior

ProWarrior — Change Notes

New: Arms Swing Timer

A dedicated weapon swing timer bar for Arms spec, designed around the Slam cast window.

• Shows only during active combat when spec is set to Arms
• Bar depletes from full (just swung) toward zero (next swing imminent)
• Green while remaining ≥ slamCastTime — safe to start casting Slam without clipping the auto-attack
• Transitions orange → red as the window closes
• Pulsing "SLAM!" text above the bar during the safe window
• Reads your Improved Slam talent rank at login, zone load, and on talent changes — base cast time 1.5s, rank 1 → 1.0s, rank 2 → 0.5s
• /pw swing — toggle on/off (saved per character, Arms-only note shown when on another spec)
• /pw reset resets its position; /pw unlock makes it draggable
• Default position: below the threat bar

Fix: Swing detection via CombatLogGetCurrentEventInfo()

Replaced select(N, ...) CLEU arg extraction with CombatLogGetCurrentEventInfo(), which reliably returns structured data regardless of client version. The old approach could fail if the hideCaster field was absent from the CLEU format, causing srcGUID to land on the wrong argument index and the player GUID check to never match.

Fix: TrackSwing offhand detection

WoW's CLEU passes isOffHand as integer 0 (mainhand) or 1 (offhand). Lua treats 0 as truthy, so if isOffHand then was routing every mainhand swing to ohLastSwing and mhLastSwing was never set. Fixed to isOffHand and isOffHand ~= 0, handling both boolean and integer representations.