Oh, you're right.
But actually, I dropped the system of using different members for red, green, blue, and alpha. I'm just using a single hex value as a member to save space.
Edit:
Just wondering, would this be any faster:
unsigned int red = (hex << 8) >> 16;
//or is it:
unsigned int red = (hex << 8) >> 24;