After incrementing an unsigned char value, don't cast it back to