Moderator
M
Moderator
12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.
10:19, 3rd Aug 2012
Magtheridon96: This needs a lot of work.
Instead of the wait, you should use the timer expiration event.
Also, why are you showing and hiding the timer for specific players?
Why not show it to the owner of the triggering unit instead?
By the way, instead of (Dying unit), use (Triggering unit).
It's faster and cleaner.
And instead of constantly repeating function calls like (Triggering unit) and (Last created timer dialog), store those into variables to reuse in the rest of the trigger.
Check this tutorial out to learn more about leaks because it seems that you are not clearing them.
edit
You still have leaks, and you should not be using that wait.
IcemanBo: Too long as NeedsFix. Rejected.
10:19, 3rd Aug 2012
Magtheridon96: This needs a lot of work.
Instead of the wait, you should use the timer expiration event.
Also, why are you showing and hiding the timer for specific players?
Why not show it to the owner of the triggering unit instead?
By the way, instead of (Dying unit), use (Triggering unit).
It's faster and cleaner.
And instead of constantly repeating function calls like (Triggering unit) and (Last created timer dialog), store those into variables to reuse in the rest of the trigger.
Check this tutorial out to learn more about leaks because it seems that you are not clearing them.
edit
You still have leaks, and you should not be using that wait.