-
Notifications
You must be signed in to change notification settings - Fork 12.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve std::thread::Result documentation #66114
Conversation
r? @rkruppe (rust_highfive has picked a reviewer for you, use r? to override) |
r? @Centril |
3e6b6dc
to
f1bc4ef
Compare
r=me rollup with green CI |
This comment has been minimized.
This comment has been minimized.
Shoot, it can't resolve it. Fixing. |
ce181a2
to
4317263
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
b4fd542
to
8568204
Compare
Hm, I wonder what's wrong here. According to the intra doc links RFC, this should be the correct syntax, and "not being able to resolve" sounds more of a path issue? Maybe |
Seems it worked; thanks! @bors r+ rollup |
📌 Commit 8568204 has been approved by |
…entril Improve std::thread::Result documentation Thanks to @dtolnay for pointing out the different premise of the contents of the `Err` variant in `std::thread::Result` WRT normal error handling.
Rollup of 12 pull requests Successful merges: - #65794 (gate rustc_on_unimplemented under rustc_attrs) - #65945 (Optimize long-linker-command-line test) - #66044 (Improve uninit/zeroed lint) - #66076 (HIR docs: mention how to resolve method paths) - #66084 (Do not require extra LLVM backends for `x.py test` to pass) - #66111 (improve from_raw_parts docs) - #66114 (Improve std::thread::Result documentation) - #66117 (Fixed PhantomData markers in Arc and Rc) - #66146 (Remove unused parameters in `__thread_local_inner`) - #66147 (Miri: Refactor to_scalar_ptr out of existence) - #66162 (Fix broken link in README) - #66171 (Update link on CONTRIBUTING.md) Failed merges: r? @ghost
Thanks to @dtolnay for pointing out the different premise of the contents of the
Err
variant instd::thread::Result
WRT normal error handling.