[Solved] Quick Question

[Jupiterstorm]

Hey, it's been a while since I did any work on a map and I wanted to ask a quick question.

Does this leak?

• Collect Vis
  • Events
    • Map initialization
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 10, do (Actions)
      • Loop - Actions
        • Set PlayerVisamene[(Integer A)] = (PlayerVisamene[(Integer A)] + (Number of units in (Units owned by Players[(Integer A)] matching ((Unit-type of (Matching unit)) Equal to |cff0066ccVisamene Extractor|r))))
        • Set PlayerVisamene[(Integer A)] = (PlayerVisamene[(Integer A)] + ((Number of units in (Units owned by Players[(Integer A)] matching ((Unit-type of (Matching unit)) Equal to |cff0066ccVisamene Extractor|r (Charged)))) x 2))
    • Wait 10.00 seconds
    • Trigger - Run (This trigger) (checking conditions)

 

[Xonok]

A unit group if anything.

Whenever you count "units matching condition" it creates a group. So you can either use "set bj_wantDestroup = true" before every time you use a group. Or you can store the group and destroy later.

 

[deathismyfriend]

if u dont have any conditions u should run trigger ignoring conditions. i would also change the event over since its running every 10 seconds. change it to periodic timer w 10 second interval. and for each integer A u should make ur own variable since the for each integer A is slower than ur own variable and it can cause some really weird bugs when used w some things

 

[Almia]

He asked "Does this leak?"
You will answer just yes or no.

Yes

 

[Jupiterstorm]

Thanks guys. Very helpful

 

[Rapid]

Yess