A Problem with DisableTrigger

cleeezzz

The Undead Ranger.
Reaction score
268
JASS:
   call DisableTrigger(LazorTrig)
    loop
        set fog = FirstOfGroup(g)
        exitwhen fog == null
        call UnitDamageTarget(d.tower,fog,d.damage,false,false,ATTACK_TYPE_NORMAL,DAMAGE_TYPE_UNIVERSAL,null)
        set d.damage = d.damage * .75
        call GroupRemoveUnit(g,fog)
    endloop
    call EnableTrigger(LazorTrig)


those are the actions, and these are the events,

JASS:
private function Init takes nothing returns nothing
    local trigger trig = CreateTrigger(  )
    set LazorTrig = trig
    call TriggerRegisterAnyUnitDamaged(trig)
    call TriggerAddAction( trig, function Actions )
endfunction


yet it still infinite loops O_O
 
Try commenting out the EnableTrigger() call to see if that changes anything. I'm not sure really what my logic behind that is...but it doesn't hurt to try. Also, add a healthy amount of DebugMessages to track what its doing.
 
tried the boolean as well, it still pases o_O, (note that this function is a callback function of the Main Actions)

edit: even commenting out the enable doesn't do anything O_OOOO
 
Are you sure it's an infinite loop problem (or if it is, that it's caused by that)?
 
oh wait, my custom event

JASS:
private function Call takes nothing returns boolean
    local integer i = COUNT - 1
    loop
        if TriggerEvaluate(TRIGS<i>) then
            call TriggerExecute(TRIGS<i>)
        endif
        set i = i - 1
        exitwhen i &lt; 0
    endloop
    return false
endfunction</i></i>


that probably calls the trigger no matter if its disabled or not.. how do i add a check to see if trigger is disabled?
 
just to point out something, in your init function, u can just set Lazortrig = CreateTrigger(), don't need to assign it to local var then to Lazortrig. Even if you want to assign it to local var, I suggest you set trig= null at the end to avoid leak.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • V-SNES V-SNES:
    Happy Friday!
    +1
  • The Helper The Helper:
    News portal has been retired. Main page of site goes to Headline News forum now
  • The Helper The Helper:
    I am working on getting access to the old news portal under a different URL for those that would rather use that for news before we get a different news view.
  • Ghan Ghan:
    Easily done
    +1
  • The Helper The Helper:
    https://www.thehelper.net/pages/news/ is a link to the old news portal - i will integrate it into the interface somewhere when i figure it out
  • Ghan Ghan:
    Need to try something
  • Ghan Ghan:
    Hopefully this won't cause problems.
  • Ghan Ghan:
    Hmm
  • Ghan Ghan:
    I have converted the Headline News forum to an Article type forum. It will now show the top 20 threads with more detail of each thread.
  • Ghan Ghan:
    See how we like that.
  • The Helper The Helper:
    I do not see a way to go past the 1st page of posts on the forum though
  • The Helper The Helper:
    It is OK though for the main page to open up on the forum in the view it was before. As long as the portal has its own URL so it can be viewed that way I do want to try it as a regular forum view for a while
  • Ghan Ghan:
    Yeah I'm not sure what the deal is with the pagination.
  • Ghan Ghan:
    It SHOULD be there so I think it might just be an artifact of having an older style.
  • Ghan Ghan:
    I switched it to a "Standard" article forum. This will show the thread list like normal, but the threads themselves will have the first post set up above the rest of the "comments"
  • The Helper The Helper:
    I don't really get that article forum but I think it is because I have never really seen it used on a multi post thread
  • Ghan Ghan:
    RpNation makes more use of it right now as an example: https://www.rpnation.com/news/
  • The Helper The Helper:
  • The Helper The Helper:
    What do you think Tom?
  • tom_mai78101 tom_mai78101:
    I will have to get used to this.
  • tom_mai78101 tom_mai78101:
    The latest news feed looks good

      The Helper Discord

      Members online

      No members online now.

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top