"Constants" at least thats what i think they all are

[Dois]

constant unittype UNIT_TYPE_STRUCTURE                   = ConvertUnitType(2)

this thing that appears grey in JassCraft... UNIT_TYPE_STRUCTURE

they do that... so as an example...

IsUnitType(GetFilterUnit(), UNIT_TYPE_STRUCTURE)

can I instead ddo

IsUnitType(GetFilterUnit(), ConvertUnitType(2))

and thats better right? Or is there an even better way!

anyway just asking...

Those constants... I can go ahead and throw it away and use the ConvertUnitType(2) like that right?

 

[Magentix]

Vexorian's Optimizer fills in constants for you, the things you should replace are usually BJ functions, not constants.

 

[Sooda]

> ConvertUnitType(2)

I assume these (UNIT_TYPE_STRUCTURE for example) variables are set when map loads. Instead calling each time ConvertUnitType(x) you use global variable what is constant because once set it dosn' t change.

 

[Joker(Div)]

I'm pretty sure it doesn't matter. If you optimize your map with Vex's optimizer, it inlines all constants

 

[Dois]

ok thats nice to know

thanks guys

 

[SFilip]

Using UNIT_TYPE_STRUCTURE is faster - ConvertUnitType(2) means you do a function call every time the game runs that line. And a function call is generally slower than using a variable.
Constants exist for a reason. By all means, use them.

 

[Dois]

oh.. so it would be better to do...

local unit U = GetTriggerUnit()

at the start of the trigger and use "U" all the way instead of using GetTriggerUnit() for all the functions thats gonna "take" the unit?

 

[Tinki3]

Exactly.

 

[Dois]

ok thanks!