change this name, "bytes" does not help with determining the underlying type. it also needs a _t at the end. TBH, the underlying type is short enough to not need a typedef, and you can easily infer what a std::vector<char> is for.

It should also be char, not unsigned char.