[vJass] Error - "Function redeclared"

[Carl-Fredrik]

Hey again... the second trigger I made worked on first try (Yay!), the second one is causing a mess though.

First one:

scope Revive initializer Init
function Conditions takes nothing returns boolean
    return IsUnitType(GetTriggerUnit(), UNIT_TYPE_HERO) == false
endfunction

function Actions takes nothing returns nothing
    local location UnitLoc = GetUnitLoc(GetTriggerUnit())
    call PolledWait(10)
    call CreateNUnitsAtLoc( 1, GetUnitTypeId(GetTriggerUnit()), GetOwningPlayer(GetTriggerUnit()), UnitLoc, bj_UNIT_FACING )
    set UnitLoc = null
    call RemoveLocation (UnitLoc)
endfunction

//===========================================================================
function Init takes nothing returns nothing
    local trigger trig = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( trig, EVENT_PLAYER_UNIT_DEATH )
    call TriggerAddCondition( trig, Condition( function Conditions ) )
    call TriggerAddAction( trig, function Actions )
endfunction
endscope

Second one:

scope ReviveHeroes initializer Init
function Conditions takes nothing returns boolean
    return IsUnitType(GetTriggerUnit(), UNIT_TYPE_HERO) == true
endfunction

function Actions takes nothing returns nothing
    local location HeroLoc = GetUnitLoc(GetTriggerUnit())
    call PolledWait(10)
    call ReviveHeroLoc(GetTriggerUnit(), HeroLoc, true)
    set HeroLoc = null
    call RemoveLocation (HeroLoc)
endfunction

//===========================================================================
function Init takes nothing returns nothing
    local trigger trig = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( trig, EVENT_PLAYER_UNIT_DEATH )
    call TriggerAddCondition( trig, Condition( function Conditions ) )
    call TriggerAddAction( trig, function Actions )
endfunction
endscope

Seems like I can't use the same name for the Conditions / Actions / Initializer in each trigger... I thought I could, so I just had to ask, do I have to name them different each time??

Thanks in advance!

// Carl-Fredrik

 

[AceHart]

> I thought I could

That's what the keyword "private" has been invented for.

 

[Romek]

use

private function name takes nothing returns nothing
// or
public function name takes nothing returns nothing

to make the function uncallable by another trigger/function.
(or make it require the scopename_prefix in 'public's case)

 

[Carl-Fredrik]

Oh.. forgot to make 'em private, ty for replying!

 

[Builder Bob]

your init triggers must be named

public function InitTrig takes nothing returns nothing

if you want them to be run on map init.
Also make sure the scope name is the exact same (case sensitive) as your trigger name. (the name in the list to the left)

 

[Uberplayer]

> public function InitTrig takes nothing returns nothing
It's better to use initializer and a private function named Init or so. This way the actual triggers name does not matter a shit.

 

[Builder Bob]

ah, I did not see he used initializer. Yeah, then it can be private.