[Trigger] Trigger Leak?

[cHeRoL]

Does this trigger leaks?

• Castle 1
  • Events
    • Unit - Fortress 0068 <gen> Dies
  • Conditions
    • (Unit-type of (Triggering unit)) Equal to Castle
    • (Owner of (Triggering unit)) Equal to Player 6 (Orange)
  • Actions
    • Set castle = (All allies of (Owner of (Killing unit)))
    • Game - Display to (All players) for 2.00 seconds the text: Knights have won! V...
    • Wait 2.00 seconds
    • Game - Victory Player 1 (Red) (Show dialogs, Show scores)
    • Game - Victory Player 2 (Blue) (Show dialogs, Show scores)
    • Game - Victory Player 3 (Teal) (Show dialogs, Show scores)
    • Game - Victory Player 4 (Purple) (Show dialogs, Show scores)
    • Game - Victory Player 5 (Yellow) (Show dialogs, Show scores)
    • Game - Victory Player 6 (Orange) (Show dialogs, Show scores)
    • Game - Defeat Player 7 (Green) with the message: Defeat!
    • Game - Defeat Player 8 (Pink) with the message: Defeat!
    • Game - Defeat Player 9 (Gray) with the message: Defeat!
    • Game - Defeat Player 10 (Light Blue) with the message: Defeat!
    • Game - Defeat Player 11 (Dark Green) with the message: Defeat!
    • Game - Defeat Player 12 (Brown) with the message: Defeat!
    • Custom script: call DestroyForce( udg_castle )

• Castle 2
  • Events
    • Unit - Castle 0067 <gen> Dies
  • Conditions
    • (Unit-type of (Triggering unit)) Equal to Fortress
    • (Owner of (Triggering unit)) Equal to Player 12 (Brown)
  • Actions
    • Set castle = (All allies of (Owner of (Killing unit)))
    • Game - Display to (All players) for 2.00 seconds the text: Rebels have won! Vi...
    • Wait 2.00 seconds
    • Game - Victory Player 7 (Green) (Show dialogs, Show scores)
    • Game - Victory Player 8 (Pink) (Show dialogs, Show scores)
    • Game - Victory Player 9 (Gray) (Show dialogs, Show scores)
    • Game - Victory Player 10 (Light Blue) (Show dialogs, Show scores)
    • Game - Victory Player 11 (Dark Green) (Show dialogs, Show scores)
    • Game - Victory Player 12 (Brown) (Show dialogs, Show scores)
    • Game - Defeat Player 1 (Red) with the message: Defeat!
    • Game - Defeat Player 2 (Blue) with the message: Defeat!
    • Game - Defeat Player 3 (Teal) with the message: Defeat!
    • Game - Defeat Player 4 (Purple) with the message: Defeat!
    • Game - Defeat Player 5 (Yellow) with the message: Defeat!
    • Game - Defeat Player 6 (Orange) with the message: Defeat!
    • Custom script: call DestroyForce( udg_castle )

 

[Emm-A-]

no it doesnt leak.
You could maybe convert it to custom script and change the Conditions-Function to something easier.
But this would be jass.

Anyways its not important as your game is finsihed after this trigger, so why do you care? ^^

Greets

 

[cHeRoL]

But,I have put this system Death System then Castle 1 and Castle 2 won't work.
When I kill Fortress or Castle the game will not be finished.

 

[Emm-A-]

ah ok, then you could Remove the Dying Unit by putting the
custom script: call RemoveUnit(GetDyingUnit())
so that the Dying unit will be removed from the game.

 

[cHeRoL]

I must change the variable?

 

[Squiggy]

ah ok, then you could Remove the Dying Unit by putting the
custom script: call RemoveUnit(GetDyingUnit())
so that the Dying unit will be removed from the game.

Does dying unit still work after a wait?

 

[cHeRoL]

If I kill a Hero (Player 6 or 12 wich has Castle and Fortress) destroy all Player 6/12 bases and nothing happens,and this is not owrite.
I think that Death System fault,I will delete it,pitty,I like it .

 

[Emm-A-]

Does dying unit still work after a wait?

hm im not sure, as i code in jass with locals.... maybe TriggeringUnit is better as it is local :=)

 

[GhostWolf]

Did you like get a REALLY hard hit on your head? You state in the event a SPECIFIC unit, then you CHECK if it's that unit? You don't need those conditions...
As to that useless group... well, it's useless, as you do not even use it.

 

[cHeRoL]

Thanks to all,I will try again and again.

EDIT: Thanks to you GhostWolf,I have deleted those conditions and it works.