[JASS] Is this Jass correct?

[Fundo!]

As in will it work??

call CreateUnitAtLoc (GetOwningPlayer(GetSpellAbilityUnit()),  'h00O', udg_LocPoint3, AngleBetweenPoints(udg_LocPoint1, udg_LocPoint2)) returns unit
    local unit u = CreateUnitAtLoc( (GetOwningPlayer(GetSpellAbilityUnit()),  'h00O', udg_LocPoint3, AngleBetweenPoints(udg_LocPoint1, udg_LocPoint2)) returns unit
    call UnitApplyTimedLife(u,  'BTLF', 3.00)
    call RemoveLocation( udg_LocPoint1 )

 

[Arnadath]

i dnt see any mistake. I suggest you remove all 3 udg_LocPoint ecxept if they are static

 

[bReathl3sS]

you must skip the return things at the end of the line, they are just an information for you what type will be returned or whether there will be returned nothing. this means you can't set a variable to a function that returns nothing.

cleaned code:

    local unit u = CreateUnitAtLoc(GetOwningPlayer(GetSpellAbilityUnit()), 'h00O', udg_LocPoint3, AngleBetweenPoints(udg_LocPoint1, udg_LocPoint2))
    call UnitApplyTimedLife(u, 'BTLF', 3.00)
    call RemoveLocation(udg_LocPoint1)
    //if loc 2 and 3 are not static:
    call RemoveLocation(udg_LocPoint2)
    call RemoveLocation(udg_LocPoint3)

 

[swipe5weep]

if i remember it right.
declaration of locals should be done before calling functions.

 

[Adiktuz]

you call create unit first then you create another one via the local so your are making 2 units..... you should remove the first line... and swipe5weep is right locals are only supported on the top of actions...