Moderator
M
Moderator
7 November 2015
Bribe: Rejecting due to the status of this resource being "needs fix" for years.
13:05, 22nd Jan 2010
TriggerHappy:
Your spell is 90% jass, so just use jass instead of lame GUI. This also means I am going to review it like a Jass spell.
First off, remove all the BJ's, including GetLastCreatedUnit which can be replaced by bj_lastCreatedUnit.
I would highly suggest using timers over TSA's as well.
Furthermore, nothing is configurable so copying this spell over will be a pain and not to mention that its a very simply and overdone spell. And because of that it probably won't get approved.
But we will see.
Bribe: Rejecting due to the status of this resource being "needs fix" for years.
13:05, 22nd Jan 2010
TriggerHappy:
Your spell is 90% jass, so just use jass instead of lame GUI. This also means I am going to review it like a Jass spell.
First off, remove all the BJ's, including GetLastCreatedUnit which can be replaced by bj_lastCreatedUnit.
I would highly suggest using timers over TSA's as well.
Furthermore, nothing is configurable so copying this spell over will be a pain and not to mention that its a very simply and overdone spell. And because of that it probably won't get approved.
But we will see.