[JASS] What is wrong with this [JASS] function (different one :P)?

[xXMephistoXx]

I'm making an AoE spell that pushes all units within range of the caster away. I think this function should do the trick, but nothing happens. Nothing at all.

function QuakeFunction takes nothing returns nothing
    local unit c = GetTriggerUnit()
    local unit t
    local real x1 = GetUnitX(c)
    local real y1 = GetUnitY(c)
    local real x2 = GetUnitX(t)
    local real y2 = GetUnitY(t)
    local location pc = GetUnitLoc(c)
    local location pt = GetUnitLoc(t)
    local real dist = 300
    local real angle = Atan2(y2 - y1, x2 - x1)
    local real dur = 1
    local real radius = 100
    local integer effects = 0
    local string effectmdl = ""
    local string effectpoint = ""
    local group g = GetUnitsInRangeOfLocAll((150 * GetUnitAbilityLevelSwapped(SPELL2_ID, c)), pc)
    call GroupRemoveUnit(g, c)
    set bj_forLoopAIndex = 1
    set bj_forLoopAIndexEnd = CountUnitsInGroup(GetUnitsInRangeOfLocAll(300.00, pc)) - 1
    loop
        exitwhen bj_forLoopAIndex > bj_forLoopAIndexEnd
        set bj_forLoopAIndex = bj_forLoopAIndex + 1
        set t = GroupPickRandomUnit(g)
        //Knockback Effect
        call KnockbackEx(t, dist, angle, dur, radius, effects, effectmdl, effectpoint)
        call GroupRemoveUnit(g, t)
    endloop
    //Leak Fixes
    set c = null
    set t = null
    set pc = null
    set pt = null
endfunction

PS: "SPELL2_ID = A001"

What could be wrong with it?

 

[Dr Super Good]

It leaks firstly.
CountUnitsInGroup(GetUnitsInRangeOfLocAll(300.00, pc)) - 1

Secondly try a process of debugging.
You place debug messages in the script to find which parts work and which do not. Once you find the broken part you try and fix it.

 

[xXMephistoXx]

How does it leak, and how do I run a debug process?

 

[Dr Super Good]

Because it makes a group that you do not destroy?

You add code to display text at certain parts of your script. If the code does not display, then that part is not working (it does not run that code), if they all do display, you know you are not doing something right with certain parts of the script.

 

[xXMephistoXx]

I see =)
Fixed the leak, and I also figured out the problem. Since I'm not setting the local "t" in the beginning, I can't set things that uses "t". You bet I feel smart now..