| > Could that replace std::collections::hash_map as the main supported map crate? No, it is more specialized for certain kinds of loads. > It imports std::collections::hash_map, which has unsafe code, but only for RandomState, which does not. If RandomState were pulled out of hash_map and moved to a crate with no unsafe code, that dependency could be removed and it would be safe crates all the way down. That's a very vacuous distinction. This is all in the same crate (std) anyway, it's just a different module. > The author of the code doesn't get to determine "unnecessary". The auditors (proposed in this blog post) do. I've audited unsafe in the past to ensure our dependencies are fine, there are some pretty reasonable ways to define "unnecessary" here. |