No u shouldn't destroy that group coz it didn't Create it ... or set it ...
I had a long chat me and Dark_Dragon about it loool..
Also it will not leak like i told u it is SET or Create
About Set CO_Off[CO_Index[2]] = True
Well look in Comandership Loop... at first ... and see when it turns off ..
The Loop will do nothing ... it like to prevent bugs
Glad to help .. now it is FULLY MUI without any bugs ..
lol xD was not long at all
anyway i checked ur spell as u wanted Deuterium + i checked Rmx one...
k thats what i wanted to tell u it was not MUI until Rmx added that boolean (true / false) thing...
this is coz you always loop to max possible counter but i suggest u to decrease counter and use actaually one one. however that would require some 'smart swap algorithms' so ill leave u at this as is coz its cool
k Rmx now its my turn to blame u for not founding leaks xD
-
Unit - Create 1 SFXDummy_CO for (Owner of (Triggering unit)) at (CO_CenterLoc[CO_Index[2]] offset by (CO_MaxDistance[CO_Index[2]] / 2.00) towards 60.00 degrees) facing 150.00 degrees
... already 6 point leaks
this is in main trigger
... but forget 6 leaks xD
this is in loop trigger
-
Unit - Move CO_SFXDummy1[CO_Index[3]] instantly to (CO_CenterLoc[CO_Index[3]] offset by (CO_MaxDistance[CO_Index[3]] / 2.00) towards (CO_EffectOffsetCounter[CO_Index[3]] + 60.00) degrees), facing (CO_EffectOffsetCounter[CO_Index[3]] + 150.00) degrees
:O thats too much
so lets see how much points does he leak per cast:
the spell lasts 30 secs.
so its: 6*(30/0.03+1) =
= 6 *(100+1)
= 606 //
:O thats way tooo much xD
so as u can see everyone makes mistakes but must say gj on making it fully MUI for Deuterium since this is his first spell lol! and its quite good
so Deuterium ill suggest u to use Rmx version and fix this leaks i pointed + if u really want u can do that what -BZK- said, then it'll be great
EDIT: Actually to explain groups!
what GUI does when u create global 'Unit Group' variable is that it generates function which is executed on map init before everything else and it sets that variables to this:
JASS:
set udg_MyGroupVar = CreateGroup()
the creategroup is the only way to create group. when you set the group vars GUI does not have CreateGroup but it has advanced functions which use CreateGroup functions such as Pick every unit matching... or any of this will generate a new group. so if you are not changing the group but only removing and adding units there is no need to remove that group since you did not lost sight of it and you will again use it. but points well once you use them you cant move them and use again so thats why we remove the points. groups dont need to be destroyed if you will use it again (the same one).
Hope it helps!
Greets all!
~Dark Dragon