Can you find the leak?

[newtonrox]

function filter_Copy takes nothing returns nothing
    local integer i = 0
    local unit u = GetFilterUnit()
    local real x = GetUnitX(u)
    local real y = GetUnitY(u)
    if GetUnitTypeId(u) == 'h001' and GetPlayerState(GetOwningPlayer(u), PLAYER_STATE_RESOURCE_FOOD_USED) < 70 then
        loop
        exitwhen i >= 5
            call CreateUnit(GetOwningPlayer(u), 'h00C', x,y, bj_UNIT_FACING)
            set i = i + 1
        endloop
    endif
    set u = null
endfunction

function Actions takes nothing returns nothing
    local group g = CreateGroup()
    call GroupEnumUnitsInRect(g, bj_mapInitialPlayableArea, null)
    call ForGroup(g, function filter_Copy)
    call DestroyGroup(g)
    set g = null
endfunction

//===========================================================================

function InitTrig_Spawn_Basic_Copy takes nothing returns nothing
    set gg_trg_Spawn_Basic_Copy = CreateTrigger()
    call TriggerRegisterTimerEventPeriodic(gg_trg_Spawn_Basic_Copy, 2.0)
    call TriggerAddAction(gg_trg_Spawn_Basic_Copy, function Actions)
endfunction

cuz i can't and this is crashing my game after 2 hours XD
this is a footmen frenzy spawn-like trigger
pls help me
(when i disable this trigger it does not crash )

 

[TriggerHappy]

call GroupEnumUnitsInRect(g, bj_mapInitialPlayableArea, null)

Null filters leak.

 

[Renendaru]

call GroupEnumUnitsInRect(g, bj_mapInitialPlayableArea, null)

Null filters leak.

It's technically not a leak, but it takes up memory or something like that. Just use an empty function to return true or whatever.

 

[newtonrox]

wait XD
i'm just starting to learn filters.
so i don't exactly what to do. can you post the right code pls? ^^
thx

 

[TriggerHappy]

function thefilter takes nothing returns boolean
    return true
endfunction

function actions..
    call somefunc(...,..., Filter(function thefilter))
endfunction

 

[newtonrox]

ok thx
but "thefilter" is not my "filter_Copy" right?

 

[Renendaru]

No...?

 

[Exide]

call DestroyGroup(g)

Yes, I know it sounds crazy.

Group-less enumerating is your friend:
http://www.thehelper.net/forums/showthread.php?t=132926

Don't use a local group variable. Creating and destroying temporary groups everytime the trigger runs is a lot of useless work. -Use a global variable.