Clean up game_ui.c and rework compiler flags + fix the warnings they caused. (#556)

* warns prelim

* Update Makefile

* catch a rain straggler

* wowie

* m

* game_ui.c moment

* Update game_ui.c

* patch

* format

include/level_object_entries.h

@@ -887,7 +887,7 @@ typedef struct LevelObjectEntry_Unknown127 {
 
 typedef struct LevelObjectEntry_Hud {
     LevelObjectEntryCommon common;
-    s32 unk8;
+    s32 offsetY;
 } LevelObjectEntry_Hud;
 
 typedef struct LevelObjectEntry {

include/macros.h

@@ -47,7 +47,7 @@
 #endif
 
 // Use built-in pseudocode where possible in NON_MATCHING builds
-#ifdef __GNUC__
+#if defined(__GNUC__) && !defined(CC_CHECK)
 #define abs(x)      __builtin_abs(x)
 #define fabsf(x)    __builtin_fabsf(x)
 #define sqrtf(f)    __builtin_sqrtf(f)
@@ -75,7 +75,7 @@
 #define INCONSISTENT 
 
 // Used to make a u32 colour value look clearer. Transforms 0xFF0000FF to 255, 0, 0, 255
-#define COLOUR_RGBA32(r, g, b, a) (((r << 24) | (g << 16) |  (b << 8) | a))
+#define COLOUR_RGBA32(r, g, b, a) ((u32)((r << 24) | (g << 16) |  (b << 8) | a))
 
 // A few systems in the game use an array as a cache table. This gives you the asset ID
 #define ASSETCACHE_ID(x)    ((x << 1) + 0)