Rampage, Flurry, and Unbridled Wrath should be handled as generic "hit effects"

[vigo2]

I'd propose something along

interface Buff {
   ApplyOrRefresh(duration, simTime, simState)
   MaybeRemove(simTime, simState)
}

interface Proc {
   OnHit(hitState, simTime, simState)
}

with the various supported procs and buffs/debuffs being simple script-like implementations. This allows to remove a ton of global or near global ("manager") state, keeps the various "swing()"-like methods clean, and removes the runtime overhead of perpetually checking for non-existing procs or buffs.