[Trigger] Trigger Help

[Superclaude]

I need a trigger that does like
A hero dies
and it brings on a message:
(Player Name) has been killed by (Player Name) and has earned 500 gold.
Gives 500 gold to triggering player.

 

[MistAssass]

• Heroes
  • Events
    • Unit - A unit Dies
  • Conditions
    • ((Dying unit) is A Hero) Equal to True
  • Actions
    • Set Victim = (Dying unit)
    • Set Killer = (Killing unit)
    • Set V_player = (Owner of Victim)
    • Set K_player = (Owner of Killer)
    • Game - Display to (All players) the text: ((Name of V_player) + ( has been killed by + ((Name of K_player) + and for that he has earned 500 gold.)))
    • Player - Set K_player Current gold to ((K_player Current gold) + 500)
    • Set Victim = No unit
    • Set Killer = No unit

 

[-Berz-]

That trigger should work. MistAssass you are becoming really helpful =)

 

[MistAssass]

ty BerZeKeR but still that has a leak ... ok 2 ... i do not know how to remove player variables ...

 

[-Berz-]

set udg_<variable> = null

 

[MistAssass]

Ty for the help

 

[-Berz-]

But this is just for nulling stuff (units, integers..)

 

[Airandius]

• Heroes
  • Events
    • Unit - A unit Dies
  • Conditions
    • ((Dying unit) is A Hero) Equal to True
  • Actions
    • Set Victim = (Dying unit)
    • Set Killer = (Killing unit)
    • Set V_player = (Owner of Victim)
    • Set K_player = (Owner of Killer)
    • Game - Display to (All players) the text: ((Name of V_player) + ( has been killed by + ((Name of K_player) + and for that he has earned 500 gold.)))
    • Player - Set K_player Current gold to ((K_player Current gold) + 500)
    • Set Victim = No unit
    • Set Killer = No unit

You don't need to store all those units, they do not leak.

 

[SanKakU]

yeah i think pretty much all event response stuff doesn't leak. although they might have error if you chose the wrong event response.

 

[Dr Super Good]

Globals that are used more than once do not leak. . .
It is only locals that you have to worry about nulling.

Yes they might mean that a handle index is not recycled for quite some time, but if it gets used again it will eventually be recycled. The problem is with local handles as they dissapear after the function finishes executing its code and so can leak permantly if you do not set them to null.