Leak Variable Conflict?

[bumbaclo]

So I recently discovered the method of removing leaks from actions and such, but I have a question that I want to ask the more experienced JASSers:

I have this leak-fixed trigger:

• Trapping
  • Events
    • Time - Every 5.00 seconds of game time
  • Conditions
  • Actions
    • Set Temp_Group = (Units owned by Player 1 (Red) of type Unfathomable Trap)
    • Unit Group - Pick every unit in Temp_Group and do (Actions)
      • Loop - Actions
        • Unit - Cause (Picked unit) to damage (Random unit from (Units within 800.00 of (Position of (Picked unit)) matching (((Owner of (Matching unit)) is in GoodForce) Equal to True))), dealing 1000000000.00 damage of attack type Chaos and damage type Normal
    • Custom script: call DestroyGroup( udg_Temp_Group)

What I'm wondering is suppose I have many other similar triggers, but running on a different periodic timer - say one every 1, 5, and 10 seconds. At every 10 seconds of game time they are all going to fire at exactly the same moment. If thats the case, wouldnt using a global variable (Temp_Group) malfunction in some way since it can't be a temporary group for all the periodic events at the same time?

 

[Komaqtion]

Well, all of the triggers fire on a different "thread", and the processor only works on a single thread at a time, meaning that it'll execute the triggers one by one so nothing will get overwritten unless you use a wait somewhere in the trigger (There is a VERY small chance, but it's close to none )

And, just for your information, that trigger still leaks 2 handles...
One unit-group and a location

 

[bumbaclo]

Lol, thanks for telling me, forgot about groups in groups. Is this fixed:

Edit: realized it was completely off, will continue to work on it =p

Also, would it be possible to convert those global variables into local ones to prevent the chance of overlap from happening?

 

[jig7c]

the above trigger will not work because

• Unit Group - Pick every unit in Temp_GroupArray[(Integer A)] and do (Actions)
  • Loop - Actions
    • Unit - Cause (Picked unit) to damage (Random unit from Temp_GroupArray[(Integer A)]), dealing 1000000000.00 damage of attack type Chaos and damage type Normal
    • Custom script: call DestroyGroup( udg_Temp_GroupArray[GetForLoopIndexA()])

is outside of the integer A loop..

 

[jig7c]

Also, would it be possible to convert those global variables into local ones to prevent the chance of overlap from happening?

yes

private function $name$ takes nothing returns nothing
local group temp_group
local group array temp_grouparray

sorry for double posting
got so caught up in helping the fellow, forgot i was dp (double posting)