will this desync?

Pay special attention to the enum part, this is called inside a ForGroup....

My definition is desyncers is anything which will cause data to mess up. local special effects don't change any gameplay constants (units, terrain height, etc.) and therefore won't cause any of the game data to be different to any players. Sspecial effects can be seem but nothing else; it has no body, collision, etc. Therefore, I conclude that changing the color, although seen by one player only, does not effect gameplay constants. But of course I could be wrong in everything I'm saying and your probably testing it at this very moment.

I am pretty sure that will not desync. It's merely a graphical interface.

Only object data (doodads, units, items, unit orders etc) will cause desyncs.

It should be fine. The If comparison will happen for all computers (calling the GetEnumUnit() ).

The function inside the if is fine since this is the same principle the AllyColor buttons works (the one beside the minimap).

wont desync.

I tested it on bnet and it did not desync. case closed.