Skip to content

Conversation

pietroalbini
Copy link
Member

@pietroalbini pietroalbini commented May 13, 2018

r? @alexcrichton

eddyb and others added 2 commits May 13, 2018 16:20
If check_expr_struct_fields fails, do not continue to record update.
If we continue to record update, the struct may cause us to ICE later
on indexing a field that may or may not exist.
@rust-highfive
Copy link
Contributor

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 13, 2018
@alexcrichton
Copy link
Member

@bors: r+

Thanks @pietroalbini!

@bors
Copy link
Collaborator

bors commented May 13, 2018

📌 Commit 7041b65 has been approved by alexcrichton

@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 May 13, 2018
@pietroalbini
Copy link
Member Author

@bors p=1 (beta backport)

@bors
Copy link
Collaborator

bors commented May 13, 2018

⌛ Testing commit 7041b65 with merge 5d261a0...

bors added a commit that referenced this pull request May 13, 2018
[beta] Process backports

* #50622: rustc: leave space for fields of uninhabited types to allow partial initialization.
* #50643: typeck: Fix ICE with struct update syntax

r? @alexcrichton
@bors
Copy link
Collaborator

bors commented May 13, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 5d261a0 to beta...

@bors bors merged commit 7041b65 into rust-lang:beta May 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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.

6 participants