Daelin (25/60) ¯¯¯¯¯¯ Coding: 9/18 Errors at Martyrism_GetTarget, final is not initialized correctly and it may crash the thread if it doesn't find enough enemies, thus it then is unable to destroy the group and boolexpr, also in Martyrism_Main the calls to DestroyGroup and DestroyBoolexpr are in the wrong place so it might call DestroyGroup/Boolexpr on initialized variables and stop the function from getting to the nulling part. Radiance leaks a triggeraction. Beacon uses a check for ethereal unit type, wouldn't it be better to just use a magical damage type? Not important: Bloodmana's spellsteal protection limits its levels at 3 without an option for max levels in the header so it fails following the Jesp standard. * The movement on inisidius beacon is not the best way, it is better to use some math thus preventing the usage of Atan2, Sin and Cos. x,y=x2-unitx,y2-unity , r=wantedmovedist/sqrt(x*x+y*y) unitx=x1+x*r, unity=y1+y*r. Could even replace sqrt(x*x+y*y) with the remaining distance from gamecache. Votes: 16/42 Moyack (24/60) ¯¯¯¯¯¯ Coding: 12/18 Land Bash: Why use a local string array for the effects? Seriously. It doesn't make much sense to make a variable for the Boolexpr if you don't plan to destroy it. GetUnitsInRangeOfLocMatching leaks a group reference, Checking current order is not exactly the best method for detecting the end of channeling spells. Votes: 12/42 Rising Dusk 21/60 ¯¯¯¯¯¯¯¯¯¯¯ Coding 14/18 - Not important: Although the spells claim that it follows the JESP standard, they don't . "Handle variables" must follow the codename. - Triggers leak, they are disabled instead of Destroyed, and calls to timer stuff are always used with I2Timer, it would seem that saving it to an integer would save some time. But these seem to be safety measures. - Needles uses SetUnitPosition instead of SetUnitX/Y , that would optimize the movement a lot, since there are a lot of needles. Votes: 7/42 Pyrogasm 9/60 ¯¯¯¯¯¯¯¯ Coding: 2/18 (sorry but it is full of leaks) - GetUnitsInRangeOfLocMatching (and the GUI equivalent) leaks a group reference. - There's an special effect leak in Planar Rift if the unit was already ethereal. - Planar Disruptance\Spell cast also has got this kind of leaks. - Defabricate comes with a massive group leak every 0.3 seconds! - Also 2 special effect leaks. Votes : 7/42 Remaining contestants: ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯ emjlr3 1 vote shadowange1 1 vote Zachary_Shadow 0 votes Argo Disqualified ToadCop Disqualified