knickish
knickish
Fixed the clippy workflow to work with the new combinations and ran rustfmt
Any interest in merging this or should I just use my fork?
@seladb Sure, will see what I can do about making it smaller this weekend.
@seladb put up another pr https://github.com/seladb/pickledb-rs/pull/31 for the part that's just fixing the feature flags, after that one is merged this one should just add the nanoserde changes. Sorry I...
Closed in light of discussion in #31
#70 is up to potentially implement this, wanted to get feedback on whether it is worth a breaking change with a version bump.
Varint encoding definitely would save some space for the binary format at least, but I'm a little concerned about the performance overhead all the additional branches would cause. I would...
results from [rust_serialization_benchmark](https://github.com/djkoloski/rust_serialization_benchmark) on this branch vs. master: ``` log/nanoserde/serialize time: [171.92 µs 172.05 µs 172.21 µs] change: [-0.0851% +0.4384% +0.8537%] (p = 0.07 > 0.05) No change in performance...
> However, personally, I would still prefer to have them all by default this is still the case in this PR, all of them are on by default, so hopefully...
Made `nanoserde_derive` an optional dep so that `toml` can be used without it