Moderator
M
Moderator
07:43, 13th Aug 2013
Magtheridon96: There are memory leaks in here.
Doomlord's Review
You may want to have another trigger that runs on map initialization with the purpose of setting the spell constants like the special effect path, the attachment point, etc...
This makes it easier for users to modify a spell without having to touch any of the code that actually executes the spell.
Check this out as well.
Purge: Although the memory leaks have been fixed, I feel like this spell is a bit too simple. Don't take it as offensive though, it is definitely a step in the right direction.
The only issue is that this spell has too little complexity to be accepted as useful. I'm sorry. ;(
Magtheridon96: There are memory leaks in here.
Doomlord's Review
You may want to have another trigger that runs on map initialization with the purpose of setting the spell constants like the special effect path, the attachment point, etc...
This makes it easier for users to modify a spell without having to touch any of the code that actually executes the spell.
Check this out as well.
Purge: Although the memory leaks have been fixed, I feel like this spell is a bit too simple. Don't take it as offensive though, it is definitely a step in the right direction.
The only issue is that this spell has too little complexity to be accepted as useful. I'm sorry. ;(