search

fitzgen / bumpalo

A fast bump allocation arena for Rust
https://docs.rs/bumpalo
Apache License 2.0
1.36k stars 111 forks source link

Reduce crate size #120

Closed lrazovic closed 2 years ago

lrazovic commented 2 years ago

Using the include field in the Cargo.toml 23% (or 100.9 KB) can be saved in 18 files (of 433.6 KB and 33 files in entire crate). The final .crate file goes from 131KB to 76.5 KB.

fitzgen commented 2 years ago

I'd prefer to use exclude rather than include, just so that it is less likely that I forget to include some file or directory in the future that is now required because of some recent change and then break the whole package anyone depending on the crate. I'd prefer the failure mode where I forget to add new excludes and the packages gets a tiny bit more bloated than it should be, rather than breaking things for users.

Happy to merge if you flip this around into an exclude!

lrazovic commented 2 years ago

Flipped from include to exclude, with the same size decrease. Now from the .crate are excluded tests, benchmarks and especially the logo.