Leak problem

[bessomi]

Teleport
    Events
        Unit - A unit owned by Player 1 (Red) Finishes casting an ability
    Conditions
        (Ability being cast) Equal to Teleport (Neutral Hostile)
    Actions
        Set Teleport[(Player number of (Owner of (Casting unit)))] = (Rally-Point of Fat shop 0002 <gen> as a point)
        Unit - Move (Target unit of ability being cast) instantly to Teleport[(Player number of (Owner of (Casting unit)))]
        Custom script:   call RemoveLocation( udg_Teleport )

Teleport is a point array. So i think it expects an array in the custom script, what do i do?

 

[Artificial]

Custom Script:    call RemoveLocation(udg_TempPoint[GetPlayerId(GetOwningPlayer(GetTriggerUnit())) + 1])

But if you only got that one event, the player is always going to be red, isn't it? So it's kinda stupid to get the Owner of (Casting unit).

 

[bessomi]

I have up to 11 players.

Does the code change?

And do i need to change tempoint to teleport?

 

[Artificial]

Does the code change?[/code]If you mean: Does that work for all of them? Then yes, it does.

And yeh, you need to change it to Teleport (sry, I misread it ).

 

[AceHart]

Edit, Convert to custom text, and have a look at what "player number of ..." looks like in JASS...

Then again, if "teleport" is not needed outside this trigger, then using an array there is pointless.

 

[bessomi]

I am guessing what he wrote is the exact same thing.

But may i ask why there is a + 1 in the end?

 

[Romek]

I am guessing what he wrote is the exact same thing.

But may i ask why there is a + 1 in the end?

In JASS, Player numbers start from 0. So Player 1 is Player(0) in JASS.
That + 1 just makes it easier to use that global in other non-Jass triggers