Skip to content

Fixup some nits from #44238#44329

Merged
bors merged 1 commit into
rust-lang:masterfrom
nrc:pulldown-warn-fix
Sep 10, 2017
Merged

Fixup some nits from #44238#44329
bors merged 1 commit into
rust-lang:masterfrom
nrc:pulldown-warn-fix

Conversation

@nrc

@nrc nrc commented Sep 5, 2017

Copy link
Copy Markdown
Member

@nrc

nrc commented Sep 5, 2017

Copy link
Copy Markdown
Member Author

cc #44229

@arielb1 arielb1 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 5, 2017

@ollie27 ollie27 left a comment

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.

Apart from that one error this looks good to me.

Comment thread src/librustdoc/html/render.rs Outdated

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.

This should be 45 to make sure the resulting string is at most 70 characters.

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.

Addressed this, thanks! I set to 50 (as in the old code), there is no strict need to be under 70 chars, it is just roughly a number that should fit on one line (comfortably under 80 chars).

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.

Well that's okay but if the line is exactly 71 characters for example then using 50 will skip one character and replace it with " ... " making the string display actually longer. It's only a minor thing though.

@nrc nrc force-pushed the pulldown-warn-fix branch from f0a816d to 0573949 Compare September 5, 2017 20:35
@nrc

nrc commented Sep 6, 2017

Copy link
Copy Markdown
Member Author

ping @ollie27 for re-review

@ollie27

ollie27 commented Sep 7, 2017

Copy link
Copy Markdown
Contributor

@bors: r+

@bors

bors commented Sep 7, 2017

Copy link
Copy Markdown
Collaborator

@ollie27: 🔑 Insufficient privileges: Not in reviewers

@nrc

nrc commented Sep 7, 2017

Copy link
Copy Markdown
Member Author

@bors: r=ollie27

@bors

bors commented Sep 7, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 0573949 has been approved by ollie27

@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors rollup

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Sep 9, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Sep 9, 2017
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Sep 10, 2017
bors added a commit that referenced this pull request Sep 10, 2017
Rollup of 13 pull requests

- Successful merges: #44262, #44329, #44332, #44347, #44372, #44384, #44387, #44396, #44449, #44451, #44457, #44464, #44467
- Failed merges:
@bors bors merged commit 0573949 into rust-lang:master Sep 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants