SC2Mapster

Thx. Link to the official blizzard website added in the main post.

Yes, you need to keep the InitTrig... function but yes even in ROC you can convert the gui trigger in full jass2.

What about suggest this on the official blizzard forum, it's really needed, it is not some fashion quite useless need. http://forums.battle.net/board.html?forumId=26951354&sid=5010

I can't, i don't have a beta key :(

It seems that i can't use them correctly, could you edit my post plz ? Also have a sticky post which are and how to use tags would be good, or at least i have not found this thread.

EDIT : Oh ok , all are detailed inside a wiki, when you click on "Click here for details", when you are writing a new post or editing.

I'm entirely with you here, but no ROC didn't allow custom script section it was a TFT feature.

First, i will compare galaxy with jass2, the language script of the warcraft3 editor.

- I love your name convention for function, it's much more consistent than jass2. With jass2 we couldn't really guess the function name, sometimes it began with the manipulated type, and sometimes not. For example, CreateTimer/TimerStart for wc3 and TimerCreate/TimerStart for Galaxy.

- I understand that you try to make generic functions instead of X functions, but sometimes we miss functions.

• For example

native unitgroup UnitCreate (
    int inCount,
    string inUnitType,
    int inCreateStyle,
    int inPlayer,
    point inPos,
    fixed inFacing
);

Seriously most of the time we only need to create one unit, and even if we need several ones, we could simply use a loop. So it's kinda annoying have to use an unitgroup just for one unit ... Since you already use the basic name UnitCreate, could i suggest UnitCreateSingle, or something like that ?

• It also lacks UnitGetX, and UnitGetY, and in general functions which use X/Y coordinates instead of a point.

• There is no "player" type, and instead Galaxy use int, that's not bad but sadly there are many functions which use a playergroup but not an int. For example

  native void     DialogSetVisible (int dialog, playergroup players, bool isVisible);
  

- We need a function PlayerIdLocal (or ofc a better name) which return a different int on each human computer, the local player id.

- Not something really relevant but for const int you should use "-1" only for invalid input only and "0" for default input to avoid errors.

Link to the blizzard official website : http://forums.battle.net/thread.html?topicId=24401857344&sid=5010