Rather than looping over an indexed table on every aura event, just use a hash table instead:
Code:
local bloodlust = { [2825]=1, [32182]=1, [90355]=1, [160452]=1, [80353]=1, [1459]=1 }
Then you can just do a simple table lookup inside the event handler:
Code:
if bloodlust[spellId] then
(Note that the value of each entry being 1 is irrelevant; any non-nil/false value will do. 1 is just nice and short.)
Also, looking up UnitName("player") on every aura event is pretty wasteful. Just do it once and store it in a variable outside of the event handler:
Code:
local PLAYER = UnitName("player")
...then refer to that variable in the event handler:
Code:
if (type == "SPELL_AURA_APPLIED") and (destName == PLAYER) then