self and
event are always passed with the event, the rest varies from event to event.
Take UNIT_AURA for example, it passes "unitid" as well as those mentioned above [1]. But you have to catch it yourself nowadays.
So your example would either be (only showing the relevant bits):
Code:
OnEvent:SetScript("OnEvent", function(self, event, ...)
if event == "UNIT_AURA" then
local unit = select(1, ...)
if unit == "target" then
<snip>
or
Code:
OnEvent:SetScript("OnEvent", function(self, event, unit, ...)
if unit == "target" then
<snip>
Either way is fine, but the latter basically assumes you've only registered for the UNIT_AURA event (or any other event with that particular signature) while the first can be used for other events as well.
[1]
http://wowprogramming.com/docs/events/UNIT_AURA