Should probably be an unsigned rather than an unsigned char.  Doesn't really matter.