valence
valence copied to clipboard
The `Text` struct is too large.
cargo clippy
often complains that std::mem::size_of::<Text>()
is too large. There are a few things that should be done.
- Stop using
Cow<'static, str>
everywhere and just useString
. We can reintroduce it later if it turns out to be important. - Get rid of the
Option<bool>
s by manually implementingSerialize
andDeserialize
. Use a bitset with two bits perOption<bool>
.
I'd like to take a stab at this.