Problem with leaks

[Lightskin]

Well, the problem is basically I don't understand how to detect leaks, and could someone please tell me where are the leaks here? As I'm pretty sure it leaks, since it laggs out the game, but, dunno where :S Or it doesn't leak?

• Spawn Wild Animals
  • Events
    • Time - Every 10.00 seconds of game time
  • Conditions
  • Actions
    • Unit - Create 1 Panther for Neutral Hostile at (Random point in (Playable map area)) facing (Random angle) degrees

• Not At Water
  • Events
    • Unit - A unit enters (Playable map area)
  • Conditions
  • Actions
    • Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Terrain type at (Position of (Triggering unit))) Equal to (==) Sunken Ruins - Sand
      • Then - Actions
        • Unit - Move (Triggering unit) instantly to (Random point in (Playable map area))
        • Trigger - Run (This trigger) (checking conditions)
      • Else - Actions

 

[Axarion]

Both triggers leak some point variables and the first on uses very much randomness which isn't very performant. Also the second trigger runs himself?! You should read some tutorials about leaks

 

[Lightskin]

Yes, because the point is that it moves the unit until it isn't in the sand.

 

[The_DooM]

Hm...the first trigger should only leak because of the random point. You could do this: Set RandomPoint(variable)=RandomPointPlayableArea then Create 1 Panther at RandomPoint and then Custom Script: call RemoveLocation(udg_RandomPoint). Similar could be used for the second one. Hope it helps

 

[Berb]

What The DooM said is correct, you need to store the "random" point as a variable so that it can explicitly be removed (and thus recycled) after the variable is no longer necessary.

 

[Lightskin]

Well, many thanks, it now looks like:

• Spawn Wild Animals
  • Events
    • Time - Every 100.00 seconds of game time
  • Conditions
  • Actions
    • Set RandomPoint = (Random point in (Playable map area))
    • Unit - Create 1 Panther for Neutral Hostile at (Random point in (Playable map area)) facing (Random angle) degrees
    • Custom script: call RemoveLocation(udg_RandomPoint)

• Not At Water
  • Events
    • Unit - A unit enters (Playable map area)
  • Conditions
  • Actions
    • Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Triggering unit)) Equal to (==) Panther
        • (Terrain type at (Position of (Triggering unit))) Equal to (==) Sunken Ruins - Sand
      • Then - Actions
        • Set RandomPoint = (Random point in (Playable map area))
        • Unit - Move (Triggering unit) instantly to (Random point in (Playable map area))
        • Custom script: call RemoveLocation(udg_RandomPoint)
        • Trigger - Run (This trigger) (checking conditions)
      • Else - Actions

This one shouldn't leak, right?

Thanks once again.

 

[watermelon_1234]

You're supposed to use the variables.
Ex:

• Unit - Create 1 Panther for Neutral Hostile at RandomPoint facing (Random angle) degrees

 

[Lightskin]

Haha, forgot the most important thing!! xDD Thanks watermelon_1234.

 

[The_DooM]

This one shouldn't leak, right?

Thanks once again.

Yes, it shouldn't if you fixed what Watermelon said. np for the help Anyway, if you have anyother trigger that has random points, use the same for them to remove the leaks. Just one more thing, the first trigger in your older post says every 10sec, in your newer it says every 100sec, might be just a miss type but still check it. Hope it helps

 

[Lightskin]

Oh yeah, thanks for the alert, actually it says 10 sec because of testing, I wouldn't wait 100 secs for a test
And the second says 100 because is the real time between each spawn.