Fix color24

[briaguya-ai]

fixes #352

not sure if you had a different way to handle this in mind @Kenix3 but imo this is way better than having it be straight up broken

[Malkierian]

Could you do an auto on the grabbing of the variable from variable->second and keep the clr.r/g/b intact for the rest of it and have it work?

[briaguya-ai]

@Malkierian no, Color and Color24 are separate members of the cvar struct

libultraship/src/config/ConsoleVariable.h

Lines 13 to 21 in 6671d7e

	typedef struct CVar {
	const char* Name;
	ConsoleVariableType Type;
	int32_t Integer;
	float Float;
	std::string String;
	Color_RGBA8 Color;
	Color_RGB8 Color24;
	} CVar;

if we wanted to do a bigger rework we could change that struct to have

typedef struct CVar {
    const char* Name;
    ConsoleVariableType Type;
    int32_t Integer;
    float Float;
    std::string String;
    uint8_t ColorR;
    uint8_t ColorG;
    uint8_t ColorB;
    uint8_t ColorA;
} CVar;

and then just return a Color or Color24 using those on Get or something

[Kenix3]

Not super thrilled with the complicated ternaries, but I don't like the fact that we have a struct on the cvar at all, so fixing that feels out of scope.