Should be uint8_t ?