Moved the '++' for cleaner code

HurricanGame / Hurrican

Freeware jump and shoot game created by Poke53280, based on the Turrican game series by Manfred Trenz

62 stars 22 forks source link

Moved the '++' for cleaner code #33

Closed Mia75owo closed 2 years ago

Mia75owo commented 2 years ago

This really bothered me for some reason. It changes nothing on how the code works but i think it just looks better. This is basically the lowest priority a pr can have...

thrimbor commented 2 years ago

I'm not particularly fond of the code as a whole - moving the operator is more or less a cosmetic change, but the underlying issue is that this table is initialized at runtime at all. Making HighscoreColors a static const(expr) std::array is a better choice imho

drfiemost commented 2 years ago

Marked the array as static const as suggested above so this can be closed. Thanks for bringing it up.