Скилл написан с нуля на чистом jass'е, есть вопросы - пишите, всегда отвечу

Как скопировать твою Спелы я не понил как скопировать Трейгере скопировал 3 тригера потом волк пехотинца в свою карту скопировал потом не знаю по конкретне не можеш писать или этот спел зделан другой прогой ?

Слишком часто вызывается GetHandleId(t), занеси её в переменную.

Вместо:
local real x = GetLocationX(GetUnitLoc(a))+60*Cos(angle*bj_DEGTORAD)
надо бы:
local real x = GetWidgetX(a)+60.*Cos(angle*.017)

Вместо:
local real angle = bj_RADTODEG * Atan2(GetLocationY(GetSpellTargetLoc()) - GetLocationY(GetUnitLoc(a)), GetLocationX(GetSpellTargetLoc()) - GetLocationX(GetUnitLoc(a)))
это:
local real angle = 57.295*Atan2(GetSpellTargetY() - GetWidgetY(a), GetSpellTargetX()- GetWidgetX(a))

Вместо:
set bj_lastCreatedUnit = CreateUnitAtLoc(GetOwningPlayer(a), YourDummy, Location(x,y), angle)
вот это:
set bj_lastCreatedUnit = CreateUnit(GetTriggerPlayer(), YourDummy, x, y angle)

И вместо ForGroup() лучше бы циклом пробегаться.

и лучше использовать для сохранения юнитов и эффектов call SaveAgentHandle(), а не те, что сейчас. вместо GetSpellAbilityUnit() можно GetTriggerUnit()
мелочи жизни, но вроде быстрей работает)

И не совсем понятно, зачем делать так: set dammy = bj_lastCreatedUnit, ведь ты юнита уже определил. дальше незачем в новую переменную заносить)

И странно, что таймер в функции Trig_spell_Actions запускается раньше, чем на него сохранятся данные:) да и интервала таймера .04, в целом, достаточно. .01 слишком часто, мне кажется. хотя, это уже кому как нравится)

RectContainsLoc(bj_mapInitialPlayableArea, Location(x1,y1)) желательно бы раскрыть)

function condfordead takes nothing returns boolean
local timer t = GetExpiredTimer()
local unit dammy = LoadUnitHandle(hash,GetHandleId(t),1)
if(not( GetUnitState(dammy, UNIT_STATE_LIFE) <= 0 == true)) then
return false
endif
return true
endfunction
- лучше вообще убрать и перейти на перебор юнита циклом, а не через ForGroup(), а то тут и таймер висячий остаётся и его хендл не очищается.

кстати, вообще не видел действия FlushChildHashtable(). Получается, у тебя после того, как прошли все действия с таймером, его хендл в памяти торчит и засоряет её.

да, и вместо:
function Trig_Die_debug_Conditions takes nothing returns boolean
if ( not ( GetUnitTypeId(GetDyingUnit()) == YourDummy ) ) then
return false
endif
return true
endfunction

это:
function Trig_Die_debug_Conditions takes nothing returns boolean
return GetUnitTypeId(GetTriggerUnit()) == YourDummy
endfunction

в общем, переходи на координаты, не используй точки, не используй ForGroup(), раскрывай бж, оптимизируй код. удачи)

рекомендую почитать статьи про оптимизацию кода на хгм. там хоть и старенькие они, но кое-чем помогут.

Он ведь писал, что нужно поменять равкоды на свои.