Moderator
M
Moderator
12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.
19:24, 28th Oct 2012
Magtheridon96:
IcemanBo: Too long as NeedsFix. Rejected.
19:24, 28th Oct 2012
Magtheridon96:
- The total axes should be configurable in the initialization trigger
- The configuration should support multiple levels.
- (Casting unit) -> (Triggering unit)
- You're leaking a location inside the Cast trigger:
- Unit - Create 1 Axe Spiral Unit for (Owner of AS_Caster[AS_Current_Index]) at (AS_Point offset by AS_AxeDistance towards ((360.00 / (Real(AS_Total_Axes[AS_Current_Index]))) x (Real(AS_Integer))) degrees) facing Default building facing degrees
- (AS_Point offset by AS_AxeDistance towards ((360.00 / (Real(AS_Total_Axes[AS_Current_Index]))) x (Real(AS_Integer))) degrees)
This location is also being leaked inside the Loop trigger. - Your spell is not MUI because AS_Timer and AS_Height must be arrays. AS_Timer should not be decreased at the top outside of the loop, you must decrease it inside the loop for each instance after making it an array.
- When you pick units in the "Hit" group, the 100.00 radius should be configurable, not constant.