Skip to content

Add missing urls and improve internal doc representation#37804

Closed
GuillaumeGomez wants to merge 0 commit into
rust-lang:masterfrom
GuillaumeGomez:net_urls
Closed

Add missing urls and improve internal doc representation#37804
GuillaumeGomez wants to merge 0 commit into
rust-lang:masterfrom
GuillaumeGomez:net_urls

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

Comment thread src/libstd/net/ip.rs
/// - test addresses used for documentation (192.0.2.0/24, 198.51.100.0/24 and 203.0.113.0/24)
/// - the unspecified address (0.0.0.0)
///
/// [ipv4-sr]: http://goo.gl/RaZ7lg

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any idea why this is a shortened url? Do we have a way to break up long URLs?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By clicking on it? :p

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To rephrase my previous question: Why don't we just put the target URL here instead of the shortened URL? Not a big deal though, just curious

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe it's to please make tidy's line length check. We should find a better way though. If we add ignore-tidy-linelength to the file then we can use the full URL but then the whole file won't be checked.

Comment thread src/libstd/net/ip.rs Outdated
/// [ip7][http://man7.org/linux/man-pages/man7/ip.7.html]
/// W. Richard Stevens, p. 891; see also [ip7].
///
/// [ip7]: (http://man7.org/linux/man-pages/man7/ip.7.html)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

std/net/struct.Ipv4Addr.html:63: broken link - std/net/(http:/man7.org/linux/man-pages/man7/ip.7.html)

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@frewsxcv

Copy link
Copy Markdown
Contributor

Just kidding, wasn't delegated for this one. r=me anyways.

@steveklabnik

Copy link
Copy Markdown
Contributor

@frewsxcv give me one second 😉

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@GuillaumeGomez

Copy link
Copy Markdown
Member Author

@bors: delegate=frewsxcv

@GuillaumeGomez

Copy link
Copy Markdown
Member Author

Damn... Let's do it the old way.

@bors: r=frewsxcv rollup

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r-

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r-

(Sorry for the spam)

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@GuillaumeGomez

Copy link
Copy Markdown
Member Author

@bors: r=frewsxcv

@bors

bors commented Nov 17, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 21f577f has been approved by frewsxcv

@GuillaumeGomez

Copy link
Copy Markdown
Member Author

Finally! \o/

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors p=-1000

@frewsxcv

Copy link
Copy Markdown
Contributor

@bors rollup-

@frewsxcv

Copy link
Copy Markdown
Contributor

Woooo, it's working!

@bors rollup

@bors

bors commented Nov 17, 2016

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #37837) made this pull request unmergeable. Please resolve the merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants