How??

[GrandMaitre]

Hi I have a simple question about action in trigger.
How to kill every units IN A REGION please???
Imposible to find...

 

[Nichilus]

Create a unit group, using "Pick every unit in region" and then when you iterate through the unit group use "Unit - kill (Picked unit)"

 

[Wietlol]

Dont use "Unit - Kill (Picked unit)".
Use "Unit - Add a generic expiration timer of 0.01 seconds" instead.

Also be sure to check out the leaks section if you are not aware of unit group leaks.

 

[Zwiebelchen]

Dont use "Unit - Kill (Picked unit)".
Use "Unit - Add a generic expiration timer of 0.01 seconds" instead.

Huh? Why?

 

[Wietlol]

In short: "Unit - Kill <unit>" leaks.
(Which means that KillUnit() also leaks.)

 

[Zwiebelchen]

In short: "Unit - Kill <unit>" leaks.
(Which means that KillUnit() also leaks.)

Can you back that up with a source?

 

[Wietlol]

-_-

Try it yourself.

function test1 takes nothing returns nothing
    local unit u = CreateUnit(Player(0), 'hfoo', 0, 0, 0)
    
    call RemoveUnit(u)
    
    set u = null
endfunction
function test2 takes nothing returns nothing
    local unit u = CreateUnit(Player(0), 'hfoo', 0, 0, 0)
    
    call KillUnit(u)
    call RemoveUnit(u)
    
    set u = null
endfunction

globals
    unit u2 = null
endglobals
function test3 takes nothing returns nothing
    call RemoveUnit(u2)
    
    set u2 = CreateUnit(Player(0), 'hfoo', 0, 0, 0)
    call UnitApplyTimedLife(u2, 0, 0.01)
endfunction


function InitTrig_Test takes nothing returns nothing
    call TimerStart(CreateTimer(), 0.011, true, function test1)
endfunction

The first one only suffers very little memory leaks because of the creation of a unit.
There is no known way to go around it.

The second one is increasing the RAM usage massively (compared to the first).

The third one is a bit different because removing the unit before the expiration timer expired is not going to do much so you have to do it with the last unit.
But that one suffers from the same leaks as the first.