Delete unused pub struct Color.

[kpreid]

Another cleanup that is technically a breaking change. The struct was completely unused inside the crate and unusable outside.

An alternative to this PR would be to use Vec<Color> in place of the returned Vec<u8>, but that would require several more impls to be really usable.

[bschwind]

LGTM - let me know if you have any more breaking changes on the way, I was close to publishing 0.2 on crates.io but I'll hold off for a bit if there are more changes we want to make soon.

[kpreid]

I have one more in mind — what do you think of making the return type be Vec<[u8; 4]> so that it is statically proven to be a multiple of 4 bytes, but still flexible? (Users wanting Vec<u8> can use Vec::into_flattened() to get that.)

I won’t be doing any more PRs today, though — time for me to sleep.

[bschwind]

I have one more in mind — what do you think of making the return type be Vec<[u8; 4]> so that it is statically proven to be a multiple of 4 bytes, but still flexible?

I think that makes sense, but let me think it through a bit before we go with it.