[JASS] UnitApplyTimedLife -> not removing units completly?

[Zacharias]

I´m currently recognizing serious performance problems in my map. After testing several functions i found the problem. Sinces i use a buff system that needs timed dummy units (with Aloc), those wont get removed after time is up. They where removed from screen (the model) but can still be found e.g. by GroupEnumUnitsOfPlayer. As number of units in map reaches 500 it gets laggy...

So two questions:

1. Is it just an error of my code/thoughts or really the truth, that units with timed life wont be removed completly?
2. How to fix it? I need to place a buff without using sleeps or timed units.

 

[xD.Schurke]

first of all try it with 'BTLF' instead of 'Aloc' and don't forget to null the locals or globals

 

[Zacharias]

first of all try it with 'BTLF' instead of 'Aloc' and don't forget to null the locals or globals

Aloc - that means that i added a locust ability to them. Nulling globals or locals wont remove that units

Look, it´s just that simple:

function FleeUnit takes unit u returns nothing
    local unit uFlee = CreateUnit(GetOwningPlayer(u), 'h01C', GetUnitX(u), GetUnitY(u), 0)
    call UnitApplyTimedLife(uFlee, 'BTLF', 1)
    call IssueTargetOrder(uFlee,"slow",u)
endfunction

h01C is the dummy unit. The buff-ability is, as you can guess, the slow ability.

 

[xD.Schurke]

you forgot again to null the local... the var will take space in the memory

 

[aznricepuff]

Yea, you should fix that leak, but honestly leaking 4 bytes every time that function is called will not cause significant lag by itself after just a few hundred calls.

 

[Zacharias]

Hm nulling locals fixes leaks? Never heard of that, but thx. Will try it out.

Btw: replacing UnitApplyTimedLife by RemoveUnit will eliminate those performance issues, the units are also complety removed and cant be found by GroupEnums any more.
So someone plz tell me how to completly remove by UnitApplyTimedLife

 

[Deaod]

try changing death type to "Cannot be revived; Does not decay"

 

[Zacharias]

try changing death type to "Cannot be revived; Does not decay"

thx. Problem solved

Edit: nulling units doesn´t have any effect. It´s only neccessary for handles like locations. Tested it out by myself.