Skip to content

Conversation

estebank
Copy link
Contributor

@estebank estebank commented Apr 9, 2021

Those two recovery attempts have a very bad interaction that causes too
unnecessary output. Add a simple gate to avoid interpreting a ; as a
, when there are unclosed braces.

Fix #83498.

@rust-highfive
Copy link
Contributor

r? @lcnr

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 9, 2021
@rust-log-analyzer

This comment has been minimized.

@estebank estebank force-pushed the unclosed-brace-use branch from 6c5642d to 9be9323 Compare April 9, 2021 16:10
@rust-log-analyzer

This comment has been minimized.

…rbose

Those two recovery attempts have a very bad interaction that causes too
unnecessary output. Add a simple gate to avoid interpreting a `;` as a
`,` when there are unclosed braces.
@estebank estebank force-pushed the unclosed-brace-use branch from 9be9323 to 0d71676 Compare April 9, 2021 17:22
@jackh726
Copy link
Member

jackh726 commented Apr 9, 2021

Pretty simple :)

r? @jackh726

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 9, 2021

📌 Commit 0d71676 has been approved by jackh726

@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 Apr 9, 2021
@bors
Copy link
Collaborator

bors commented Apr 10, 2021

⌛ Testing commit 0d71676 with merge 481598b...

@bors
Copy link
Collaborator

bors commented Apr 10, 2021

☀️ Test successful - checks-actions
Approved by: jackh726
Pushing 481598b to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 10, 2021
@bors bors merged commit 481598b into rust-lang:master Apr 10, 2021
@rustbot rustbot added this to the 1.53.0 milestone Apr 10, 2021
@estebank estebank deleted the unclosed-brace-use branch November 9, 2023 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

Missing a single bracket in an import statement can cause tens of cascading errors.
7 participants