Help

[Ulla_t]

I know this is simple, but it doesent work at all.

• SlimesKillSpawn
  • Events
  • Destructible - A destructible within SlimeRegion <gen> dies
  • Conditions
  • (Destructible-type of (Dying destructible)) Equal to Demon Egg
  • (Destructible-type of (Dying destructible)) Equal to Demon Egg
  • Actions
  • Unit - Create 1 Demon (Level 1) for Neutral Hostile at (Position of (Dying destructible)) facing (Center of SlimeRegion <gen>)

 

[Accname]

The event does only work for the 64 first created destructibles in the area. This part was sloppy scripted by blizzard.
You would have to manually pick all destructibles in the area and add the event (<Picked Destructible> dies) to your trigger for each destructible picked.

You also have 2 conditions which look alike. If those are 2 different destructible types with equal names that would also be a problem since those two conditions are linked with an "and" statement and a destructible could never be of the same type as 2 different destructibles.

 

[Ulla_t]

ahh thanks

 

[Ulla_t]

Well the problem is that i use a refresh trigger, everytime a team fails killing the 'slime Eggs'(Destrictible) they will refresh (Rebuild themself). So it will always be new Eggs, so i cant pick them manually.

 

[Accname]

When creating a new one add its death event to the trigger.
This will, however, leak event handles. But those arent the "biggest" memory leaks.

 

[Ulla_t]

I cant use a 'Destructeble variable' for (Destructeble Dies)

 

[Odi]

I thought you could...

 

[GFreak45]

are you use JNGP? if you are i could write a quick library that would give you a custom script you could use to avoid that leak caused by adding hundreds of events to the same trigger

if you are this should work, create a new trigger, edit->convert to custom text, delete whats inside and paste this:

Spoiler

library EventLeakStopper
    
    globals
        private constant integer SPAWN_ID = 'hfoo'
        //This is the only thing you adjust, it is the supposed to be set to the raw data id of the unit spawned
        //replace the 'hfoo' with '[id of demon unit type]', inside the apostraphes
        //you can get the raw data id of a unit type by finding it in the object editor, then going to View/Show Raw Data, or something to that effect
    endglobals
    
    private function Actions takes nothing returns boolean
        call CreateUnit(PLAYER_NEUTRAL_AGGRESSIVE, SPAWN_ID, GetDestructableX(GetTriggerDestructable()), GetDestructableY(GetTriggerDestructable()), 0)
        call DestroyTrigger(GetTriggeringTrigger())
        return false
    endfunction
    
    public function AddEggEvent takes destructable d returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterDeathEvent(t, d)
        call TriggerAddCondition(t, Filter(function Actions))
        t = null
    endfunction
    
endlibrary

then you can use the custom script: call AddEggEvent (udg_Destructable)

 

[GFreak45]

Reposting because i had to edit the submitted code, there was a logic error in it that would cause it to not work, if you were using it already replace it with what is there now

The library right there will actually stop leaks generated by the GUI trigger editor period, whereas every gui trigger will leak no matter what unfortunately

you call this on the eggs when they are created, set the egg to a variable then call the custom script on the variable

 

[Ulla_t]

I'm really sorry, but i fixed it yesterday, by making the 'Eggs' Into units instead, thanks for the great replies anyway!

 

[GFreak45]

lol dont worry about it, as long as it was fixed