Moderator
M
Moderator
12.12
IcemanBo: For long time as NeedsFix. Rejected.
Bribe
Set Int_Cataclysm_Player = (Owner of (Triggering unit)) -> (Triggering player)... and reference this variable instead of calling (Triggering player repeatedly)
(Casting unit) -> (Triggering unit)
(Key (Last created unit)) should be set to a variable as it involves three function calls:
When possible, avoid using arrays. Array[2], for example, can be simplified into Variable2.
Maker, v1.2:
You save the point[1] into a hashtable but remove it at the end of the cast trigger. You load it in the looping trigger, but do not use it. Fix that.
IcemanBo: For long time as NeedsFix. Rejected.
Bribe
Set Int_Cataclysm_Player = (Owner of (Triggering unit)) -> (Triggering player)... and reference this variable instead of calling (Triggering player repeatedly)
(Casting unit) -> (Triggering unit)
(Key (Last created unit)) should be set to a variable as it involves three function calls:
GetHandleIdBJ(GetLastCreatedUnit())
When possible, avoid using arrays. Array[2], for example, can be simplified into Variable2.
Maker, v1.2:
You save the point[1] into a hashtable but remove it at the end of the cast trigger. You load it in the looping trigger, but do not use it. Fix that.