std::thread: follow-up #123913 removing unsafe on success for glibc.

[devnexen]

follow up #123913

[rustbot]

r? @m-ou-se

rustbot has assigned @m-ou-se.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[Noratrieb]

hm, this is needed for unsafe_op_in_unsafe_fn, right? why does this not fail to compile 🤔, CI builders should be building this

[RalfJung]

CI builders should be building this

Should they? That lint is denied in alloc/src/lib.rs, but in std it's only denied in some modules and sys::pal is not one of them.

[RalfJung]

follow up #123913

That's a rollup. Which of the PRs in there do you mean?

I don't think we should do this. Instead we should set deny(unsafe_op_in_unsafe_fn) in more and more places.

[devnexen]

I was not aware of this new lint, makes sense now.

[RalfJung]

Here's the issue for using this lint in std: #73904.