[Solved] Does this leak? (Convert player to player group)

[bruunk]

Hi, I just want to make sure this doesn't leak a player group. If it leaks how would I clear it?
Not sure if it's a big deal anyways but I'm sure it's better to make sure it's clean.

And to be clear, I am clearing the point. It's the player group I'm concerned with as it says "Convert player to playergroup".

• Cinematic - Ping minimap for (Player group((Owner of (Triggering unit)))) at (Center of (Playable map area)) for 1.00 seconds

 

[Drake53]

It leaks, since you're using this function:

function GetForceOfPlayer takes player whichPlayer returns force
    local force f = CreateForce()
    call ForceAddPlayer(f, whichPlayer)
    return f
endfunction

What wouldn't leak is using the bj_FORCE_PLAYER array (this is used when you select an option from the 'Preset' dropdown in the playergroup window).
However since you need to select the player based on unit ownership, you cannot use this dropdown. Instead, you can assign a value in this array to a global variable using custom script:

• Actions
  • Custom script: set udg_playergroup = bj_FORCE_PLAYER[GetPlayerId(GetOwningPlayer(GetTriggerUnit()))]
  • Cinematic - Ping minimap for playergroup at (Center of (Playable map area)) for 1.00 seconds

In the custom script, replace 'playergroup' with the name of your global variable (make sure to keep the 'udg_' prefix).
Since you're not creating a new playergroup, you also don't have to call DestroyForce.

 

[Tasyen]

If you are in Lua mode you could overwrite the function GetForceOfPlayer to read the playergroup array instead of recreating a new force. Then you don't have to alter your GUI Trigger.

function GetForceOfPlayer(player)
    return bj_FORCE_PLAYER[GetPlayerId(player)]
end

That code you would add somewhere into your map, if you wana do that.

I think that Dsg's gc (lab section) handles player groups (force).

 

[bruunk]

Thank you both! Solved