Add missing urls in ptr docs #40703
Add missing urls in ptr docs #40703
Conversation
| @@ -255,6 +255,11 @@ mod prim_unit { } | |||
| /// | |||
| /// *[See also the `std::ptr` module](ptr/index.html).* | |||
| /// | |||
| /// [`null`]: ../std/ptr/fn.null.html | |||
There was a problem hiding this comment.
why go up and then go back into std again? that is why not
ptr/fn.null.html
There was a problem hiding this comment.
To avoid falling back on core. I had issues in the past with weird linking like this.
There was a problem hiding this comment.
Do you recall what those issues are? I know there are issues regarding linking things from libcollections because things are reexported, but wasn't aware of issues with core or std.
There was a problem hiding this comment.
I have too many PRs so I can't find it but it was common when we linked from core to not find the equivalent in core so I used to always target std. Maybe things have changed since then?
There was a problem hiding this comment.
Will this change cause issues for people who use no_std?
|
@bors: r+ rollup seems good, thanks! |
|
📌 Commit 2715101 has been approved by |
…eklabnik Add missing urls in ptr docs r? @rust-lang/docs
…eklabnik Add missing urls in ptr docs r? @rust-lang/docs
r? @rust-lang/docs