Skip to content

Commit

Permalink
Rollup merge of rust-lang#51186 - pnkfelix:remove-unneccessary-nll-st…
Browse files Browse the repository at this point in the history
…derr-files, r=oli-obk

Remove two redundant .nll.stderr files

It turns out that the diagnostics generated from NLL for these cases are now exactly the same as that produced by AST borrowck, and thus we can just fallback on those `.stderr` files that already exist for AST-borrowck.

Bravo!

(it is a good idea to remove these files, because it slightly reduces the amount of time humans will spend reviewing the .nll.stderr fileset...)

((it *might* be worthwhile trying to change the `compiletest` code to even issue a warning when two such files have equivalent contents... but I am not going so far as to try to implement that right now...))
  • Loading branch information
Mark-Simulacrum authored Jun 6, 2018
2 parents 19d0b53 + 50224ec commit 60d8ed3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 45 deletions.
11 changes: 0 additions & 11 deletions src/test/ui/generator/pattern-borrow.nll.stderr

This file was deleted.

34 changes: 0 additions & 34 deletions src/test/ui/issue-45697.nll.stderr

This file was deleted.

0 comments on commit 60d8ed3

Please sign in to comment.