Is this a leak?

[ronojales]

Hi! i have been reading into those leak guides, and though i understand that locations leak, the examples i've seen so far are only one-to-one (one unit per point). so i was wondering, this is also a leak right? (unit group and point?), and is there a way to deal with that point leak? because i only understood as far as when a single unit is ordered into a certain point, but not when many units are ordered into random points in a region.

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in Prison01 <gen> owned by Player 3 (Teal)) and do (Actions)
      • Loop - Actions
        • Unit - Order (Picked unit) to Move To (Random point in Prison01 <gen>)

*i was thinking if maybe there is a way to avoid making too many actions just to remove a single point leak.

 

[Frunck]

Yeah this leaks:

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in Prison01 <gen> owned by Player 3 (Teal)) and do (Actions)
      • Loop - Actions
        • Unit - Order (Picked unit) to Move To (Random point in Prison01 <gen>)

To fix it do this:

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Set TempGroup = Units in Prison01 <gen> owned by Player 3 (Teal)
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Unit - Order (Picked unit) to Move To (Random point in Prison01 <gen>)
    • Custom Script: call DestroyGroup (udg_TempGroup)

 

[Maker]

And the location leak:

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Set TempGroup = Units in Prison01 <gen> owned by Player 3 (Teal)
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Set TempLoc = (Random point in Prison01 <gen>)
        • Unit - Order (Picked unit) to Move To TempLoc
        • Custom Script: call RemoveLocation(udg_TempLoc)
    • Custom Script: call DestroyGroup (udg_TempGroup)

 

[ronojales]

oh, so you can include Custom Scripts inside looping actions? i mean, under the Loop - Actions line. so far what i understood was you must remove things that leak to prevent crashes, and most of the Custom Scripts i found was after the looping actions, so i had no idea until now. thanks Frunck and Maker!

 

[Frunck]

Lol I didn't look inside the unit group o.o between you don't have to clear the variable every loop you can call RemoveLocation at the end of the trigger unless you use waits inside it.

 

[Damage]

No, you must clear the location on every action inside the loop. Because for each and every action of the loop, a new point is created (Random point in the region) - each and every one of these must be removed. By putting the RemoveLocation outside of the loop, then only the last point is removed, leaking all the others.

Therefore, do exactly as Maker described.

 

[Maker]

Generally it's not a good idea to use waits inside of a loop.

And you do need to remove the location before you set new point using that variable. You definately need to remove the leak inside the loop, not at the end of the trigger.


With this you send every unit to different point.

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Set TempGroup = Units in Prison01 <gen> owned by Player 3 (Teal)
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Set TempLoc = (Random point in Prison01 <gen>)
        • Unit - Order (Picked unit) to Move To TempLoc
        • Custom Script: call RemoveLocation(udg_TempLoc)
    • Custom Script: call DestroyGroup (udg_TempGroup)

With this you send all units to the same point.

• Prisoners
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Set TempLoc = (Random point in Prison01 <gen>)
    • Set TempGroup = Units in Prison01 <gen> owned by Player 3 (Teal)
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Unit - Order (Picked unit) to Move To TempLoc
    • Custom Script: call RemoveLocation(udg_TempLoc)
    • Custom Script: call DestroyGroup (udg_TempGroup)

 

[ronojales]

yup, i wanted the first trigger Maker did, the one where units are sent to different points in the region. that was what i was thinking, if there is a way to make units go to different points inside a Unit Group loop and remove all that random locations, and not make an action for every unit and remove the points for each unit. (mhen that's 344 units total ) so i'll be sticking with what Maker suggested