Memory Leak Question

[Bloodydood]

Now since using a single point can cause leaks, yet able to be fixed like so:

• Single Point
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Set Temp_Point = (Center of (Playable map area))
    • Unit - Create 1 Footman for Player 1 (Red) at Temp_Point facing Default building facing degrees
    • Custom script: call RemoveLocation (udg_Temp_Point)

Is there a way to keep multiple points from leaking?

• Multiple Points
  • Events
    • Map initialization
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 10, do (Actions)
      • Loop - Actions
        • Set Temp_PointArray[(Integer A)] = (Random point in (Playable map area))
        • Unit - Create 1 Footman for Player 1 (Red) at Temp_PointArray[(Integer A)] facing Default building facing degrees

 

[Komaqtion]

Well... Do you need to use those Temp_PointArray points later ? :S
Or only in that trigger ?
If only in that trigger, then you don't need to use an array, just use Temp_Point and remove it inside the loop
Like:

• Multiple Points
  • Events
    • Map initialization
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 10, do (Actions)
      • Loop - Actions
        • Set Temp_Point = (Random point in (Playable map area))
        • Unit - Create 1 Footman for Player 1 (Red) at Temp_Point facing Default building facing degrees
        • Custom script: call RemoveLocation (udg_Temp_Point)

 

[Bloodydood]

• For each (Integer A) from 1 to Player_Count, do (Actions)
  • Loop - Actions
    • Set Temp_PointArray2[(Integer A)] = (Position of Hero[(Integer A)])

Player_Count being the predetermined integer of players both controlled by a User and Is Playing.

This is the problem. I want to save the positions of heroes before an event and then destroy those stored positions once the event is over and the heroes are moved back to their original places.

 

[Komaqtion]

Well, as long as you don't use them in between when setting them, and destroying them (I mean setting the variable to something else) then it's ok

 

[Bloodydood]

You're joking?

Still, is there a custom script I can use on a Point Array just in case?

 

[Komaqtion]

You're joking?

call RemoveLocation(udg_Temp_PointArray[1])

Or, if you want to use Integer A then:

call RemoveLocation(udg_Temp_PointArray[bj_forLoopAIndex])

Note: What I meant was that, yes you can use those array point variables, as long as you don't set them to something else befre destroying them

 

[Bloodydood]

call RemoveLocation(udg_Temp_PointArray[1])

Or, if you want to use Integer A then:

call RemoveLocation(udg_Temp_PointArray[bj_forLoopAIndex])

Note: What I meant was that, yes you can use those array point variables, as long as you don't set them to something else befre destroying them

Thanks

 

[TheCrystal]

Using Point Array wastes more time then having several individual TempPoints, writing up that script or finding it in a past trigger. The best thing [I would think] is to have several "Temp" things. I hope that Array has a max [below 8192], since I'm not sure how JASS and GUI variable differ (You never specify a max for an Array in JASS).