diff options
| author | bors <bors@rust-lang.org> | 2019-03-11 21:45:29 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2019-03-11 21:45:29 +0000 |
| commit | 7486b9c208f7e1c4aa1e36961927327bd793e952 (patch) | |
| tree | 64931921d75e8b07522feb967431abafc57d8fc0 /src/libstd/sys/unix/stack_overflow.rs | |
| parent | e68bf8ae15ee6c052d0bcc9252386c5c5ee86de2 (diff) | |
| parent | 07f99b9fec4ba0c549596e8e7d99553e86763d35 (diff) | |
| download | rust-7486b9c208f7e1c4aa1e36961927327bd793e952.tar.gz rust-7486b9c208f7e1c4aa1e36961927327bd793e952.zip | |
Auto merge of #59044 - petrochenkov:uiui, r=davidtwco
Filter away test annotations from UI test output
If you worked with UI tests for some time you could notice one issue affecting their readability and also readability of diffs when the tests change.
Look at the output of this test.
```rust
fn main() {
let 1 = 2; //~ ERROR refutable pattern in local binding
}
```
```
error[E0005]: refutable pattern in local binding: `-2147483648i32..=0i32` not covered
--> src/main.rs:2:9
|
2 | let 1 = 2; //~ ERROR refutable pattern in local binding
| ^ pattern `-2147483648i32..=0i32` not covered
error: aborting due to previous error
For more information about this error, try `rustc --explain E0005`.
```
You can see that the "refutable pattern in local binding" is duplicated.
One instance is the actual error, and the second instance is the expected error annotation.
This annotation is useful in the test input, but in the output it clutters the text and makes it harder to see what text refers to actual errors and what is just comments, especially if there are many errors in a single test file.
@estebank [reported](https://github.com/rust-lang/rust/pull/57379#discussion_r245523361) using the next trick to avoid the clutter
```rust
fn main() {
let 1 = 2;
//~^ ERROR refutable pattern in local binding
}
```
```
error[E0005]: refutable pattern in local binding: `-2147483648i32..=0i32` not covered
--> src/main.rs:2:9
|
2 | let 1 = 2;
| ^ pattern `-2147483648i32..=0i32` not covered
error: aborting due to previous error
For more information about this error, try `rustc --explain E0005`.
```
, i.e. using `//~^` and placing the annotation one line below will remove the annotation from the output.
However, this doesn't always works (consider errors with multi-line spans), and shouldn't be necessary in general!
`compiletest` could automatically filter away its own annotations from the output instead.
This is exactly what this PR does.
r? @davidtwco
Diffstat (limited to 'src/libstd/sys/unix/stack_overflow.rs')
0 files changed, 0 insertions, 0 deletions
