I think I heard/read somewhere (recently?) that adding all the event to one trigger may cause problems. Like if a unit dies you can't remove just one event. So it's better to keep a index for unit/trigger and deleted/make as needed.
On another note, there are already systems out there which can handle this, plus some know the difference between normal attacks and magic attacks, and their just cleaner. Leak wise and optimization wise. | 
