afisakov
You can change this now in User CP.
- Reaction score
- 37
Ok, I found a map I like and am trying to clean up some memory leaks in it to make it run smoother. The question is, did I effectively remove the leaks from this trigger and will it still do what it is supposed to do?
trigger supposed function: every 5 seconds damage every unit (not building) on the map by 80 damage for every pestilence unit that has at least 100 mana. I included the original trigger as a reference.
my version
original trigger
trigger supposed function: every 5 seconds damage every unit (not building) on the map by 80 damage for every pestilence unit that has at least 100 mana. I included the original trigger as a reference.
my version
Code:
Pestilence 2
Events
Time - Every 5.00 seconds of game time
Conditions
Actions
Set Temp_Group = (Units of type Pestilence)
Unit Group - Pick every unit in Temp_Group and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Integer((Mana of (Picked unit)))) Greater than or equal to 100
Then - Actions
Unit - Set mana of (Picked unit) to ((Mana of (Picked unit)) - 100.00)
Set Pest = (Units in (Playable map area))
Unit Group - Pick every unit in Pest and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
((Picked unit) is A structure) Equal to False
((Picked unit) is alive) Equal to True
Then - Actions
Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 80.00)
Special Effect - Create a special effect attached to the overhead of (Triggering unit) using Abilities\Spells\Undead\DeathCoil\DeathCoilSpecialArt.mdl
Special Effect - Destroy (Last created special effect)
Else - Actions
Custom script: call DestroyGroup(udg_Regen)
If ((ReverseDamage effect <gen> is on) Equal to True) then do (Unit - Set life of Mage 0001 <gen> to ((Life of Mage 0001 <gen>) - 80.00)) else do (Do nothing)
If ((ReverseDamage effect <gen> is on) Equal to True) then do (Unit - Set life of Mage 0004 <gen> to (Real(Life_before_Reverse))) else do (Do nothing)
Else - Actions
Custom script: call DestroyGroup(udg_Temp_Group)
original trigger
Code:
Pestilence Auto
Events
Time - Every 5.00 seconds of game time
Conditions
(Number of units in (Units of type Pestilence)) Greater than 0
Actions
Unit Group - Pick every unit in (Units of type Pestilence) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Integer((Mana of (Picked unit)))) Greater than or equal to 100
Then - Actions
Unit - Set mana of (Picked unit) to ((Mana of (Picked unit)) - 100.00)
Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
((Picked unit) is A structure) Equal to False
((Picked unit) is alive) Equal to True
Then - Actions
Unit Group - Add (Picked unit) to Pest
Else - Actions
Unit Group - Pick every unit in (Units of type Pestilence) and do (Actions)
Loop - Actions
Unit Group - Pick every unit in Pest and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 80.00))
Unit Group - Pick every unit in Pest and do (Special Effect - Create a special effect attached to the overhead of (Picked unit) using Abilities\Spells\Undead\DeathCoil\DeathCoilSpecialArt.mdl)
If ((ReverseDamage effect <gen> is on) Equal to True) then do (Unit - Set life of Mage 0001 <gen> to ((Life of Mage 0001 <gen>) - 80.00)) else do (Do nothing)
If ((ReverseDamage effect <gen> is on) Equal to True) then do (Unit - Set life of Mage 0004 <gen> to (Real(Life_before_Reverse))) else do (Do nothing)
Unit Group - Remove all units from Pest
Else - Actions