- Joined
- Mar 10, 2009
- Messages
- 5,014
Review for Amn's entry
- Hero model & icons are Ok except the zombie which is really not suited for this type of hero…
- Description is not well polished, I mean something like this “Then you place the bomb inside like this. See? now you throw it in and then wait for the explosion. And stay far.” <<<really bad…
- The tooltip of the hero when revived/produced/summon is missing…
- The codes are really bad, converted from GUI-Jass, and it’s not properly indented, lacks configurations also…
- You din’t filter things in ‘ug’ group properly…
- You are creating triggers & actions on every spell cast plus uses TriggerSleepAction, which I doubt that this is MUI…
- Struct impact is very bad also, it doesn’t have any members plus you’re using hashtables in it instead of ‘standard struct’, destroys triggers and uses waits also…
Rating: 2.5/10
well, I dont have againts that but it suppose to require nothing like, Alloc and LinkedListModule are independent...The_Reborn_Devil said:It's called reusing code. It's way better to make a library use one common library, than to include the library in every other library. It's why we even use libraries.
The_Reborn_Devil said:Not if they are supposed to be accessible from other libraries or scopes.
that's 'if', but this spell (particularly grip & cataclysm) wont be used in other libraries and scopes...
nerovesper said:Unfortunately though on saving KeyID's through GUI Hashtables, whenever I look for variables it won't show me any variables. I thought it was some kind of glitch or bug on the WE. at ung, one hashtable thing... yah, I just noticed. T_T
coz you're using JNGP, and it doesnt support that...use custom script "set udg_KeyId = GetHandleId(GetTriggerUnit())"
Lord_Earthfire said:I had to make a decision here. When i would add flying units into the targeting, then i would risk to cause bugs when the mechanical unit dies above unwalkable ground or above trees (in which situations the hero could get stuck). I added the cooldown to prevent a mass death/abuse of enemy mechanical units, since when the occupied mechanical unit dies, you could easiely ride another one until this one dies. I added it just against this abuse, so you could at maximum occupie 2 vehicles at once without retreating for a time.
if the hero is stuck, the kill the hero, afterall most of the crashes caused by air transport dies
well, Imo it's not an abuse coz clearly you cant use your abilities & attack
while on board, but if cooldown is really needed, the you should have put at
least 20 seconds, not 80...