nulling local variables

[storyyeller]

Which variables do you have to set to null to prevent memory leaks?

I had been doing units and groups, but I saw someone else's map set local player variables to null too. Is this necessary?

[Risen]

Every handle except real / integer should be nulled. Special ones like groups should be DestroyGroup()'d first.

[storyyeller]

I guess it's a good thing I don't use local players much then

[Aero]

I guess it's a good thing I don't use local players much then

I use local players all the time.
Locals are much better as long as you remember to null handles.

[storyyeller]

No, what I mean is that I use integers to refer to the player, since even if I used actual player variables, I'd jut have to convert to an integer whenever I wanted to index an array with it anyway

E.g. stuff like this

Code: Select all

function whatever takes nothing returns nothing
    local integer p = GetPlayerId(GetTriggerPlayer())
    // do stuff here
endfunction

[Aero]

Aha, I see. I make a lot of spells and, as you can imagine, I use a lot of unit/player variables.