it’s a data structure so it needs a fuzzer and a test oracle i’ve written a BmpVec for small bitmap-compressed vectors now i’m writing a BlimpVec which is not compressed and doesn’t have unsafe code
1
1
while i’m getting this fuzzer going, i would be very grateful if any rustaceans could express an opinion on my unsafe code - the important functions are from_cooked_parts() and as_cooked_parts() dotat.at/cgi/git/dnstrie.git…
4
yay πŸ€ͺ AddressSanitizer: double-free
1
aaaah when i overwrite myself, rust drops my old value, but instead i wanted rust to forget about it
1
1
1
fuzzer goes brrrrrrrrrrrrrrrrrrrrrrrrrr
1
they should rename libfuzzer to lovefuzzer except that might have the wrong implications wrt furries
1
1
β€œwarning: unsafe function's docs miss `# Safety` section” yes yes @dsilverstone already said that good grief
1
2
ooh a bug in clippy warning: you should consider adding a `Default` implementation for `bmpvec::BmpVec<T>` help: try this impl Default for bmpvec::BmpVec<T> { dear clippy, i think you want impl<T> there, and also i don’t want the bmpvec prefix inside mod bmpvec
2
If you can construct a minimal example of it being wrong, please file a bug. The clippy people will appreciate it.
1
hmm the warning doesn't trigger on the obvious simple example play.rust-lang.org/?version=… πŸ€·β€β™‚οΈ
1
Replying to @fanf
When you're otherwise done, point me at your codebase and I'll see if I can dig something out :D

May 13, 2021 Β· 7:01 PM UTC

1
Replying to @dsilverstone
clippy was complaining about the lack of impl Default before i added it to dotat.at/cgi/git/dnstrie.git…
2