Leak problem on spell

[slutoi]

Walking
Events
Time - Every 0.20 seconds of game time
Conditions
Actions
Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
Or - Any (Conditions) are true
Conditions
(Level of Fire Walk for (Picked unit)) Greater than 0
Then - Actions
Unit - Move dumychecker[(Player number of (Triggering player))] instantly to (Position of (Picked unit))
Else - Actions

Causing Fire
Events
Time - Every 0.10 seconds of game time
Conditions
Actions
Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Distance between (Position of (Picked unit)) and (Position of dumychecker[(Player number of (Triggering player))])) Greater than 0.01
(Level of Fire Walk for (Picked unit)) Greater than 0
Then - Actions
Unit - Create 1 Immolation Dummy for (Owner of (Picked unit)) at (Position of dumychecker[(Player number of (Triggering player))]) facing (Random real number between 0.00 and 360.00) degrees
Unit - Set Level of Permanent Immolation (Fire Walk) for (Last created unit) to (Level of Fire Walk for (Picked unit))
Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
Else - Actions
Do nothing

Learn
Events
Unit - A unit Learns a skill
Conditions
(Learned Hero Skill) Equal to Fire Walk
Actions
Unit - Create 1 Checkdummy for (Owner of (Learning Hero)) at (Position of (Learning Hero)) facing Default building facing (270.0) degrees
Set dumychecker[(Player number of (Triggering player))] = (Last created unit)
Trigger - Turn off (This trigger)

Whats wrong with it? Cuz it leaks and make my comp lag like hell. Plz help.

 

[Dr Super Good]

There is a few location leaks and group leaks but by the sound of it you are demanding too much from WC3 and so your comp is overloading. Please use TRIGGER tags around GUI to make it easier to read. . .

 

[Rui]

You're kidding, right? Two triggers running that often with leaks? Use [TRIGGER] [/TRIGGER] tags on those triggers, like DSG said, to make it easy to read.

 

[Hanky]

I think now its leakless.

• Walking
  • Events
    • Time - Every 0.20 seconds of game time
  • Conditions
  • Actions
    • Custom script: set bj_wantDestroyGroup=true
    • Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • Or - Any (Conditions) are true
              • Conditions
                • (Level of Fire Walk for (Picked unit)) Greater than 0
          • Then - Actions
            • Set location = (Position of (Picked unit))
            • Unit - Move dumychecker[(Player number of (Triggering player))] instantly to location
            • Custom script: call RemoveLocation(udg_location)
          • Else - Actions

• Causing Fire
  • Events
    • Time - Every 0.10 seconds of game time
  • Conditions
  • Actions
    • Set location2 = (Position of dumychecker[(Player number of (Triggering player))])
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
      • Loop - Actions
        • Set location = (Position of (Picked unit))
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Distance between location and location2) Greater than 0.01
            • (Level of Fire Walk for (Picked unit)) Greater than 0
          • Then - Actions
            • Unit - Create 1 Immolation Dummy for (Owner of (Picked unit)) at location2 facing (Random real number between 0.00 and 360.00) degrees
            • Unit - Set level of Permanent Immolation (Fire Walk) for (Last created unit) to (Level of Fire Walk for (Picked unit))
            • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Else - Actions
        • Custom script: call RemoveLocation(udg_location)
    • Custom script: call RemoveLocation(udg_location2)

• Learn
  • Events
    • Unit - A unit Learns a skill
  • Conditions
    • (Learned Hero Skill) Equal to Fire Walk
  • Actions
    • Set location = (Position of (Triggering unit))
    • Unit - Create 1 CheckDummy for (Owner of (Triggering unit)) at location facing Default building facing (270.0) degrees
    • Set dumychecker[(Player number of (Owner of (Triggering unit)))] = (Last created unit)
    • Custom script: call RemoveLocation(udg_location)
    • Trigger - Turn off (This trigger)

I hope I helped you.

 

[slutoi]

I dunno how to use trigger tag AND Hanky, your custom script are not working, i tried them and an "Expect a name" error message appear.

 

[Bob27]

Well if you want to find leaks in your triggers use Leak Checker. Its easy to use, and it tells you exactly how to fix the leak.