leak question

[[K40$]-Spectre]

Howdy,

Does the following trigger leak? I can't remember where I got it from, was a long time I implemented it;

• Support Bot ready
  • Events
    • Unit - A unit Finishes training a unit
  • Conditions
    • (Unit-type of (Trained unit)) Equal to Support Bot
  • Actions
    • Set TEMP_PointSoundSupportBotR = No sound
    • Custom script: if GetLocalPlayer() == GetOwningPlayer(GetTriggerUnit()) then
    • Set TEMP_PointSoundSupportBotR = PWiRdy00 <gen>
    • Custom script: endif
    • Sound - Play TEMP_PointSoundSupportBotR

Thanks!

 

[Flux]

• Support Bot ready
  • Events
    • Unit - A unit Finishes training a unit
  • Conditions
    • (Unit-type of (Trained unit)) Equal to Support Bot
  • Actions
    • Custom script: if GetLocalPlayer() == GetOwningPlayer(GetTriggerUnit()) then
    • Sound - Play PWiRdy00 <gen>
    • Custom script: endif

I'm not sure but if you want to play a sound for only one player, just do the trigger above. No variables needed.

 

[Pinzu]

• Support Bot ready
  • Events
    • Unit - A unit Finishes training a unit
  • Conditions
    • (Unit-type of (Trained unit)) Equal to Support Bot
  • Actions
    • Custom script: if GetLocalPlayer() == GetOwningPlayer(GetTriggerUnit()) then
    • Sound - Play PWiRdy00 <gen>
    • Custom script: endif

I'm not sure but if you want to play a sound for only one player, just do the trigger above. No variables needed.

This seems about right, however this:
GetOwningPlayer(GetTriggerUnit()) --> GetTriggerPlayer() Also instead of using trained unit, use TriggeringUnit.

Destroy the sound after its played (GetLastPlayedSound)

 

[Ceday]

You don't need to destroy a sound if you use the same one each time.

 

[[K40$]-Spectre]

okay thanks all