Conversation
|
r? @kennytm (rust_highfive has picked a reviewer for you, use r? to override) |
|
@rust-lang/release, is there time to squeeze this into 1.36.0? |
|
Most likely no, but I'm reaching out to folks and will try to update here if we decide to go ahead. |
src/libstd/panic.rs
Outdated
There was a problem hiding this comment.
Should there be any constraint on S?
src/libstd/panic.rs
Outdated
There was a problem hiding this comment.
I would expect the set's impl to be automatic from the map's, but that deserves a test.
There was a problem hiding this comment.
Oh, I was expected std::collections::HashSet to be based on hashbrown::HashSet but it’s not. It’s based on std::collections::HashMap, so you’re right.
|
Ideally this would be in hashbrown itself, but hashbrown is |
Fixes rust-lang#62301, a regression in 1.36.0 which was caused by hashbrown using `NonZero<T>` where the older hashmap used `Unique<T>`.
|
We've decided that we probably can go ahead with a stable CI build; I want this r+-ed before we go ahead fully though. I'll start preparing a stable-targeting PR in an hour or so. |
|
This seems good enough, we can change how we do this later, as long as @bors r+ |
|
📌 Commit 7454b29 has been approved by |
HashMap is UnwindSafe Fixes rust-lang#62301, a regression in 1.36.0-pre which was caused by hashbrown using `NonZero<T>` where the older hashmap used `Unique<T>`.
HashMap is UnwindSafe Fixes rust-lang#62301, a regression in 1.36.0-pre which was caused by hashbrown using `NonZero<T>` where the older hashmap used `Unique<T>`.
Rollup of 15 pull requests Successful merges: - #62021 (MSVC link output improve) - #62064 (nth_back for chunks_exact) - #62128 (Adjust warning of -C extra-filename with -o.) - #62161 (Add missing links for TryFrom docs) - #62183 (std: Move a process test out of libstd) - #62186 (Add missing type urls in Into trait) - #62196 (Add Vec::leak) - #62199 (import gdb for explicit access to gdb.current_objfile()) - #62229 (Enable intptrcast for explicit casts) - #62250 (Improve box clone doctests to ensure the documentation is valid) - #62255 (Switch tracking issue for `#![feature(slice_patterns)]`) - #62285 (Fix michaelwoerister's mailmap) - #62304 (HashMap is UnwindSafe) - #62319 (Fix mismatching Kleene operators) - #62327 (Fixed document bug, those replaced each other) Failed merges: r? @ghost
|
Already backported. |
|
@rust-lang/libs -- as a process note, we backported this fix to beta and stable already, but I don't think it went through official confirmation by y'all, so just pinging you to make sure it doesn't get lost |
Fixes #62301, a regression in 1.36.0-pre which was caused by hashbrown using
NonZero<T>where the older hashmap usedUnique<T>.