-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Removed the dead links from cell.rs #60539
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
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 |
@alexcrichton I am not sure what to do with this error message actually. It seems a bit unrelated to the things I did. What do you suggest on doing? |
I'm not sure I understand this PR. For me, the links seem to be working. https://doc.rust-lang.org/nightly/core/cell/struct.RefCell.html#method.try_borrow clicking on the https://doc.rust-lang.org/nightly/core/cell/struct.RefCell.html#method.try_borrow_mut clicking on the https://doc.rust-lang.org/nightly/core/cell/struct.RefCell.html#method.get_mut clicking on the
As for the original issue, the link https://doc.rust-lang.org/std/boxed/struct.Box.html#method.borrow isn't invalid per se, it's just that the methods are hidden. If you expand the methods for |
Hey @ehuss I think I am finally putting all the pieces together! I see that the issue has already been closed because of the reasons that you provided (thanks for that). I will close this PR now, since the issue is mainly due to the trait implementations not unfolding. |
Description
Removed some dead links. The code used to refer to the method
borrow
that (I think?) used to exist onBox
. However, now it is implemented by theBorrow
trait. I find nothing on theBorrow
docs that theborrow
ofBorrow
can panic as was stated in the documentation. So I removed these lines.closes #60442