Too many REMOVE LOCATION CALLS XD!
EDIT:
1st download.
EDIT2:
*Use
JASS:
set udg_X = GetUnitX ( udg_Unit )
JASS:
set udg_Y = GetUnitY ( udg_Unit )
coordinates when moving a units and do not use locations for moving the unit. You can use like this:
JASS:
set udg_Unit = GetTriggerUnit()
set udg_X = GetUnitX ( udg_Unit )
set udg_Y = GetUnitY ( udg_Unit )
set udg_MX = udg_X + udg_offset * Cos ( udg_Angle * 0.01747 )
set udg_MY = udg_Y + udg_offset * Sin ( udg_Angle * 0.01747 )
call SetUnitX ( udg_Unit , udg_MX )
call SetUnitY ( udg_Unit , udg_MY )
// Using coordinates will greatly reducing leaks and lags of the spell.
* Don't Use
-
For each (Integer A) from 1 to 6, do (Actions)
-
Loop - Actions
-
Set HB_LPUnit[(Integer A)] = (Load (Integer A) of (Key HB_Handle) in HB_Table)
because it can cause some bugs. Use your own integer variable in this.
* Use this Custom script: set udg_Handle = GetHandleId ( udg_Unit )
* If you're removing location arrays then use integer loop variable for that so it is faster than writing that remove location calls
Like this:
-
For each (Integer HB_MyOwnIntegerVariable) from 1 to 15, do (Actions)
-
Loop - Actions
-
Custom script: call RemoveLocation(udg_HB_Point[udg_HB_MyOwnIntegerVariable])
It is a quick review ^^
.
EDIT3:
I''ll rate it 3.5/5 ( because I know that this is your first spell ^^ ) but
Needs Fix
This tutorial will help you
Things a GUI User Should Know