Skip to content

Rollup of 15 pull requests - #41965

Merged
bors merged 36 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup
May 13, 2017
Merged

Rollup of 15 pull requests#41965
bors merged 36 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup

Conversation

malbarboand others added 30 commits May 5, 2017 17:12
Removes unused macros from:
* libcore
* libcollections
The last use of these two macros was removed in commit
b64c9d5
when the char_range_at_reverse function was been removed.
* librustc_errors
Their last use was removed by commits
2f2c3e1
and 11dc974.
* libsyntax_ext
* librustc_trans
Also, put the otry macro in back/msvc/mod.rs under the
same cfg argument as the places that use it.
This fix an issue where the compiler panics even if there is no
error when passed with the `-Z treat-err-as-bug` option.
Fixesrust-lang#35886.
This prevents long names from overflowing.
bootstrap: Output name of failed config in case of errors
Remove wrong or outdated info from CString docs.
None
…agisa
Refactor suggestion diagnostic API to allow for multiple suggestions
r? @jonathandturner
cc @nrc@petrochenkov
Upgrade some comments to doc comments
Fix typo in Iterator::size_hint example comment
…steveklabnik
Fix search when looking to sources
Before:
<img width="1440" alt="screen shot 2017-05-11 at 22 33 28" src="https://cloud.githubusercontent.com/assets/3050060/25970761/03fd2ade-369a-11e7-9fd6-783c23455589.png">
After:
<img width="1440" alt="screen shot 2017-05-11 at 22 34 05" src="https://cloud.githubusercontent.com/assets/3050060/25970770/09ac71ce-369a-11e7-9038-559f414e07b5.png">
r? @rust-lang/docs
rustc_trans: do not attempt to truncate an i1 const to i1.
Fixesrust-lang#41744 by skipping the truncation when it'd be a noop anyway.
Remove unused macros from the codebase
Thanks to the lint I've implemented in rust-lang#41907 I've found some unused macros inside the rustc codebase.
…Simulacrum
Add tidy check to detect empty files
Addresses rust-lang#18439.
config.toml.example: point out that optimize = false won't speed up a full bootstrap
Originally I've learned about this by @eddyb pointing this out to me over IRC, and after having told someone today the same over IRC I've thought that this is a common mistake and should be prevented by a note in config.toml.example
r? @eddyb
Fix unexpected panic with the -Z treat-err-as-bug option
This fix an issue where the compiler panics even if there is no error when passed with the `-Z treat-err-as-bug` option.
Fixesrust-lang#35886.
r? @Mark-Simulacrum
@rust-highfive

Copy link
Copy Markdown
Contributor

Some changes occurred in HTML/CSS.

cc @GuillaumeGomez

@Mark-Simulacrum

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=10

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@bors

bors commented May 13, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit f28e3cd has been approved by Mark-Simulacrum

@bors

bors commented May 13, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f28e3cd with merge cbf7560...

@bors

bors commented May 13, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@est31

Copy link
Copy Markdown
Member

The failure looks like some network error:

failed to get 200 response from `https://crates.io/api/v1/crates/utf8-ranges/0.1.3/download`, got 500

@GuillaumeGomez

Copy link
Copy Markdown
Member

@bors: retry

@bors

bors commented May 13, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f28e3cd with merge ef3ec5e...

bors added a commit that referenced this pull request May 13, 2017
@bors

bors commented May 13, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing ef3ec5e to master...

@bors
bors merged commit f28e3cd into rust-lang:masterMay 13, 2017
@Mark-Simulacrum
Mark-Simulacrum deleted the rollup branch May 13, 2017 13:59
@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

18 participants

@Mark-Simulacrum@rust-highfive@bors@est31@GuillaumeGomez@nikomatsakis@Centril@malbarbo@devurandom@mbrubeck@oli-obk@tshepang@mglagla@brson@eddyb@tommyip@Eijebong@iKevinY