Hey folks
, I'm looking for someone with expertise with unsafe code to review my new crate. GitHub - typesanitizer/concurrent-interner: A string interner that is safe to use from multiple threads
I've run the tests under MIRI and they pass (
) after about 45 minutes (
), which makes it untenable to run in CI. Happy to take feedback here or via GitHub issues. Thanks!
Hi, concurrent-interner
looks really interesting! I'll take a look at it over the next few days.
I'm far from expert, but I have done a bit of unsafe Rust code, so I'll let you know if anything jumps out at me.
One thing: there's a magic number '4192' in the code. Was it meant to be 4096?
1 Like
@typesanitizer Just FYI, I couldn't find any problems with this code. I don't know anything about panic-safety, though, so no idea if the code is OK in that respect.
Nice code, I found it very interesting to review!
1 Like
Just a minor nitpick: u16::max_value()
and usize::max_value()
are (or will be) deprecated, so they should really be replaced with u16::MAX
and usize::MAX
.