[Trigger] where does this leak?

[doom_sheep]

since adding this trigger my map has caused users to freeze in the black screen after leaving. i've also tried with the trigger disabled and am pretty sure it is the problem. someone instructed me on how to de-leak it but that does not seem to have worked as the problem persists.

• area limit
  • Events
    • Time - Every 0.09 seconds of game time
  • Conditions
  • Actions
    • Set Qunitgroup = (Units in (Entire map) matching (((Terrain type at (Position of (Matching unit))) Equal to Outland - Abyss) and (((Matching unit) has buff Voidwalker Buff ) Not equal to True)))
    • Unit Group - Pick every unit in Qunitgroup and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Picked unit)) Not equal to Gold Mine
          • Then - Actions
            • Set QUnit[1] = (Picked unit)
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (QUnit[1] is A structure) Equal to False
                • (Unit-type of QUnit[1]) Not equal to Sentry Ward
                • (Unit-type of QUnit[1]) Not equal to Sentry Ward
                • (Unit-type of QUnit[1]) Not equal to Healing Ward
              • Then - Actions
                • Set TempPoint = (Position of QUnit[1])
                • Set TempReal = (Angle from TempPoint to (Center of (Playable map area)))
                • Set QPoint[1] = (TempPoint offset by 11.69 towards TempReal degrees)
                • For each (Integer A) from 1 to 9, do (Actions)
                  • Loop - Actions
                    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                      • If - Conditions
                        • (Terrain type at QPoint[1]) Equal to Outland - Abyss
                      • Then - Actions
                        • Set QPoint[2] = (QPoint[1] offset by 6.23 towards TempReal degrees)
                        • Custom script: call RemoveLocation(udg_QPoint[1])
                        • Set QPoint[1] = QPoint[2]
                      • Else - Actions
                • Set X = (X of QPoint[1])
                • Set Y = (Y of QPoint[1])
                • Custom script: call SetUnitX(udg_QUnit[1], udg_X)
                • Custom script: call SetUnitY(udg_QUnit[1], udg_Y)
                • Custom script: call RemoveLocation(udg_TempPoint)
              • Else - Actions
                • --these will only run a handful of times per game so they don't matter
          • Else - Actions
            • --these will only run a handful of times per game so they don't matter
    • Custom script: call DestroyGroup(udg_Qunitgroup)

 

[Tasyen]

This does Leak here:
(Terrain type at (Position of (Matching unit)), it creates in each execution of the Trigger 1 Location for every unit in the Map, which is not destroyed.

Solution:
Remove it from the Filter and put it into the Group Loop as "If else".

 

[Jampion]

First line location leak: Position of Matching Unit as Tasyen already mentioned

More location leaks:

Set TempReal = Angle from TempPoint to Center of playable map area

QPoint[1] and QPoint[2] are not removed after QPoint[1] = QPoint[2]

 

[doom_sheep]

QPoint[1] and QPoint[2] are not removed after QPoint[1] = QPoint[2]

i need one those in the next iteration of the loop

 

[Tasyen]

You need to remove QPoint[1] after you repos your unit.