RoaringBitmap / roaring-rs

A better compressed bitset in Rust
https://docs.rs/roaring/
Apache License 2.0
757 stars 84 forks source link

The imports removed in `0.10.4` causes the inability to use `no_std` #278

Closed GZTimeWalker closed 5 months ago

GZTimeWalker commented 5 months ago

The deletions in the diff has destroyed support for no_std introduced in v0.10.3.

You can simply change Cargo.toml to reproduce this issue on your local:

[features]
- default = ["std"]
+ default = []

Then you'll find large number of errors caused by Vec / Sized etc.

Related issue: https://github.com/rust-lang/rust/issues/121362 https://github.com/rust-lang/rust-clippy/issues/12335 https://github.com/rust-lang/rust/issues/121708

The rust community is still working on this lint issue, as a suggestion, @kirk-baird, may you fix the errors?

cc @Kerollmops

Getting the code to work is more important than clippy's advice.