]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Rollup merge of #95591 - jackh726:nll-revisions-1, r=oli-obk
authorDylan DPC <99973273+Dylan-DPC@users.noreply.github.com>
Tue, 5 Apr 2022 20:58:58 +0000 (22:58 +0200)
committerGitHub <noreply@github.com>
Tue, 5 Apr 2022 20:58:58 +0000 (22:58 +0200)
commitcbf54fad795bc9bce7905b6deb8dfb3bac4f1e2e
treed1098ad7da7c591723443b512a8024192fb63d06
parent42ab448bb443fe792995b4aaf168c5df250a9dac
parentf881bf7c0a2c1020e36fd5f35be5ecb5af278966
Rollup merge of #95591 - jackh726:nll-revisions-1, r=oli-obk

Use revisions to track NLL test output (part 1)

The idea here is 2 fold: 1) When we eventually do make NLL default on, that PR should be systematic in "delete revisions and corresponding error annotations" 2) This allows us to look at test NLL outputs in chunks. (Though, I've opted here not to "mark" these tests. There are some tests with NLL revisions *now* that will be missed. I expect we do a second pass once we have all the tests with NLL revisions; these tests should be easy enough to eyeball.)

The actual review here should be "easy", but a bit tedious. I expect we should manually go through each test output and confirm it's okay.

The majority of these are either: 1) Only span change (the one I see most common is highlighting an entire function call, rather than just the function name in that call) 2) "E0308 mismatched types" -> "lifetime does not live long enough" 3) "E0495 cannot infer an appropriate lifetime for lifetime parameter" -> "lifetime does not live long enough" 4) "E0312 lifetime of reference outlives lifetime of borrowed content" -> "lifetime does not live long enough" 5) "E0759 `XXX` has an anonymous lifetime `'_` but it needs to satisfy a `'static` lifetime requirement" -> "lifetime does not live long enough" 6) "E0623 lifetime mismatch" -> "lifetime does not live long enough"

Other than the now lack of an error code, most of these look fine (with most giving more helpful suggestions now).

`rfc1623` output isn't great.

cc ``@marmeladema`` if you want to look through these

Let's r? ``@oli-obk`` since you've commented on the Zulip thread ;)
src/test/ui/variance/variance-btree-invariant-types.nll.stderr
src/test/ui/variance/variance-cell-is-invariant.nll.stderr
src/test/ui/variance/variance-use-invariant-struct-1.nll.stderr