Does any of this leak?

[X-Death]

Does any of this leak? If so, how and what needs cleaning?

PlayerCount is an Integer variable.

• Set PlayerCount = (Number of players in (All players matching ((((Matching player) slot status) Equal to Is playing) and (((Matching player) controller) Equal to User))))

• Animation - Play Armored Knight 0005 <gen>'s attack animation

• Game - Display to (Player group((Triggering player))) for 0.50 seconds the text: Test

• Player Group - Pick every player in (All players) and do (Actions)
  • Loop - Actions
    • Camera - Apply Cam <gen> for (Picked player) over 0.01 seconds

• Special Effect - Create a special effect at (Position of Armored Knight 0005 <gen>) using Abilities\Spells\Human\MassTeleport\MassTeleportTarget.mdl
• Special Effect - Destroy (Last created special effect)

 

[deathismyfriend]

yes anything that creates a player group.
So numbers 1 and 3

and anything that creates a point / location
so number 5

Also the player group All players should be stored in a player group variable and be used for anything in place of All players ( never destroy it or null it.

 

[X-Death]

How do I clean the leak in the variable?

 

[deathismyfriend]

look at things that leak tutorial.

 

[Daffa]

DIMF, isn't All players don't leaks?
Well, the first and last trigger leaks, being first is player group leaks and the last is the Position Leaks

 

[Maker]

All Players does not leak.

 

[Rheiko]

you should destroy your first trigger when you're done using it
and use this

• Set TempPoint = (Position of Armored Knight 0005 <gen>)
• Special Effect - Create a special effect at TempPoint using Abilities\Spells\Human\MassTeleport\MassTeleportTarget.mdl
• Special Effect - Destroy (Last created special effect)
• call RemoveLocation(udg_TempPoint)

 

[deathismyfriend]

DIMF, isn't All players don't leaks?
Well, the first and last trigger leaks, being first is player group leaks and the last is the Position Leaks

All players does not leak.

If you look at the function that matching players of whatever calls it actually creates a new player group. This is what leaks.

 

[Rheiko]

that's why we have to destroy it when we're done using it right?

 

[deathismyfriend]

Never destroy All players.

 

[Rheiko]

so how to remove that leaks?

 

[Nichilus]

so how to remove that leaks?

If you mean "All Players", you don't remove it as it does not leak. Imagine the "All players" option to be a variable created at map initialization. So every time you use All Players, you use a variable, you don't create new player group.