Rollup of 13 pull requests #36224
Closed
Rollup of 13 pull requests #36224
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
r? @steveklabnik add links for Box, Rc, and Vec
remove extra `../`s
I can no longer find these methods in nightly docs.
Previously we didn't normalize the function signatures used for closures. This didn't cause a problem in most cases, but caused an ICE in during MIR type checking. Fixes rust-lang#36139
Currently it checks only the first argument. Fixes rust-lang#31216
We've got tests which require a particular version of LLVM to run as they're testing bug fixes. Our build system, however, supports multiple LLVM versions, so we can't run these tests on all LLVM versions. This adds a new `min-llvm-version` directive for tests so they can opt out of being run on older versions of LLVM. This then namely applies that logic to the `issue-36023.rs` test case and... Closes rust-lang#36138
… r=Manishearth Add `must_use` to the Reference I'm a bit uncertain about the exact phrasing, but having it mentioned at all is probably better than before.
…teveklabnik demonstrate `RHS != Self` use cases for `Add` and `Sub`
…klabnik add links to interesting items in `std::ptr` documentation Part of rust-lang#29371. r? @steveklabnik
…teveklabnik Document try!'s error conversion behaviour try!'s documentation currently doesn't document the error conversion behaviour of the macro. This patch extends the documentation. Open questions: * is it worthwhile to have seperate examples with and without wrapping behaviour? It's not immediately obvious that From<T> for T is always defined. Though this is necessary for the macro to work in any case, is this the place to expect that knowledge.
Normalize the function signature of closures Previously we didn't normalize the function signatures used for closures. This didn't cause a problem in most cases, but caused an ICE in during MIR type checking. Fixes rust-lang#36139 r? @eddyb
…nikomatsakis Update lifetime errors to specifically note temporaries This PR updates the error message we give in the case of a temporary value not living long enough. Before: <img width="497" alt="screen shot 2016-08-31 at 10 02 47 am" src="https://cloud.githubusercontent.com/assets/547158/18138551/27a06794-6f62-11e6-9ee2-bdf8bed75ca7.png"> Now: <img width="488" alt="screen shot 2016-08-31 at 10 03 01 am" src="https://cloud.githubusercontent.com/assets/547158/18138557/2e5cf322-6f62-11e6-9047-4a78abf3d78c.png"> Specifically, it makes the following changes: * Detects if a temporary is being used. If so, it changes the labels to mention that a temporary value specifically is in question * Simplifies wording of the existing labels to focus on lifetimes rather than values being valid * Changes the help to a note, since the help+span wasn't as helpful (and sometimes more confusing) than just a note. r? @nikomatsakis
…ikomatsakis Special case a few colors for Windows As brought up on [this thread](rust-lang#33240 (comment)) the colors used in error messages on Windows can be difficult to read because of the lack of bold. This PR makes a few changes to improve readability, namely: * Rather than using BRIGHT_BLUE, on Windows we now use BRIGHT_CYAN, which is easier to read on black when you do not have bold * We used BRIGHT_YELLOW rather than YELLOW, for the same reason * Titles will be BRIGHT_WHITE now, to give the illusion of being bold Some examples:   r? @nikomatsakis cc @retep998
Transition Travis CI to use rustbuild.
configure: check if any of the arguments contain --help Currently it checks only the first argument. Fixes rust-lang#31216
test: Add a min-llvm-version directive We've got tests which require a particular version of LLVM to run as they're testing bug fixes. Our build system, however, supports multiple LLVM versions, so we can't run these tests on all LLVM versions. This adds a new `min-llvm-version` directive for tests so they can opt out of being run on older versions of LLVM. This then namely applies that logic to the `issue-36023.rs` test case and... Closes rust-lang#36138
…rner Fixed E0528 label and unit test Fixes rust-lang#36194 part of rust-lang#35233. r? @jonathandturner
…rner Fixed E0529's label and unit test Fixes rust-lang#36195 part of rust-lang#35233. This is ready for review, but will likely fail Travis due to rust-lang#36138. I changed the wording of the label, so feedback on that would be appreciated. r? @jonathandturner
…andturner Update compiler error E0558 to use new error format Fixes rust-lang#36196 part of rust-lang#35233 r? @jonathandturner
|
@bors r+ p=1 |
|
📌 Commit 5f2e32d has been approved by |
|
Tidy fail: |
|
@bors r- While I fix it |
|
@bors r+ p=1 |
|
📌 Commit 4f449e2 has been approved by |
|
⌛ Testing commit 4f449e2 with merge 84ba927... |
|
💔 Test failed - auto-linux-64-opt-rustbuild |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
must_useto the Reference #35754, demonstrateRHS != Selfuse cases forAddandSub#35793, add links to interesting items instd::ptrdocumentation #35880, Document try!'s error conversion behaviour #36099, Normalize the function signature of closures #36160, Update lifetime errors to specifically note temporaries #36171, Special case a few colors for Windows #36178, Transition Travis CI to use rustbuild. #36180, configure: check if any of the arguments contain --help #36190, test: Add a min-llvm-version directive #36198, Fixed E0528 label and unit test #36205, Fixed E0529's label and unit test #36210, Update compiler error E0558 to use new error format #36223