In-cred-ib-le

hello there little man any brother of Paladon is a brother of mine

Keep in mind that the gg_unit_H000_0000 must be generated by the world editor, otherwise it will register a unit which is "null", meaning, a variable which doesn't point at any unit at all.

function ME takes nothing returns nothing
//I never said it, but Locations are stupid. Use x/y coordinates whenever you can.
local real tX = GetOrderPointX()
local real tY = GetOrderPointY()
local string s = "Abilities\\Spells\\Other\\Doom\\DoomDeath.mdl"

//Before, you created the effect twice and never removed the first one.
call DestroyEffect(AddSpecialEffect(s, GetUnitX(gg_unit_H000_0000), GetUnitY(gg_unit_H000_0000)))
//SetUnitX/Y is also nice.
call SetUnitX(gg_unit_H000_0000, tX)
call SetUnitY(gg_unit_H000_0000, tY)
call DestroyEffect(AddSpecialEffect(s, tX, tY))

//And done. We didn't have to clear up anything this time
endfunction

function InitTrig_MEffects takes nothing returns nothing
local trigger t = CreateTrigger()
call TriggerRegisterUnitEvent(t, gg_unit_H000_0000, EVENT_UNIT_ISSUED_POINT_ORDER )
call TriggerAddAction(t, function ME )
set t = null
endfunction

Well, nothing gets "un-leaked" unless you dont destroy them, right?

Also, is is depending on the effect model you are using, when you are to destroy it.

You know ThunderClap, and WarStomp for instance. They have no specific animation index, which means, they can be removed instantly after they have been created without aborting its gfx. In cases like these, the easiest you could do is to simply:
call DestroyEffect(AddSpecialEffectLoc(s, i))

In other cases, you would have to store the effect in a variable, wait, and then destroy it.
Also, dont forget, that locations leak if you dont destroy them as well (in fact, practically everything which is not code, integer, real or boolean do leak, and are called handles)

Keep in mind, always null local handles at the end of the function.

Hey

Reputation (+1):
(Post) Thx a lot!

In
cred
id
ble