Skip to content

Polonius: update to 0.12.1, fix more move errors false positives, update test expectations #70546

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 7 commits into from
Mar 30, 2020

Conversation

lqd
Copy link
Member

@lqd lqd commented Mar 30, 2020

This PR:

  • updates polonius-engine to version 0.12.1 to fix some move errors false positives
  • fixes a fact generation mistake creating the other move errors false positives
  • updates the test expectations for the polonius compare-mode so that all (minus the 2 OOMs) ui tests pass again (matching the analysis doc starting at case 34)

In my opinion, this is safe to rollup.

r? @nikomatsakis

lqd added 7 commits March 30, 2020 01:11
This will fix some move errors false positives
This will fix the other move errors false positives:
emitting the fact at the start point caused accesses to be at the
same point as an initialization fact of the return place of a call
on the following block, which emitted an error.
…supply-region.rs

trivial diagnostics grammar change
Some impl Trait fixes lead to locating more accurately the cause of
a universal region error with a user annotation
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 30, 2020
@nikomatsakis
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Mar 30, 2020

📌 Commit 5af11d2 has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 30, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 30, 2020
Rollup of 4 pull requests

Successful merges:

 - rust-lang#70479 (avoid creating unnecessary reference in Windows Env iterator)
 - rust-lang#70546 (Polonius: update to 0.12.1, fix more move errors false positives, update test expectations)
 - rust-lang#70559 (fix BTreeMap test compilation with Miri)
 - rust-lang#70567 (Fix broken link in README)

Failed merges:

r? @ghost
@bors bors merged commit b99db6e into rust-lang:master Mar 30, 2020
@lqd lqd deleted the polonius_update branch March 30, 2020 18:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants