Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Do not attempt to suggest help for overly malformed struct/function call #91634

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

terrarier2111
Copy link
Contributor

This fixes: #91461

@rust-highfive
Copy link
Collaborator

r? @nagisa

(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 Dec 7, 2021
@nagisa
Copy link
Member

nagisa commented Dec 8, 2021

This seems good to me. Could you adjust the PR title and the commit message to describe the change in a more self-contained and immediately actionable way? e.g.

Do not attempt to suggest help for overly malformed struct/function call

would be good.

@terrarier2111 terrarier2111 changed the title Fix ICE Do not attempt to suggest help for overly malformed struct/function call Dec 8, 2021
@terrarier2111
Copy link
Contributor Author

This seems good to me. Could you adjust the PR title and the commit message to describe the change in a more self-contained and immediately actionable way? e.g.

Do not attempt to suggest help for overly malformed struct/function call

would be good.

Thanks for the review, done!

@nagisa
Copy link
Member

nagisa commented Dec 8, 2021

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Dec 8, 2021

📌 Commit b4c4bc0 has been approved by nagisa

@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 Dec 8, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 9, 2021
…askrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#87599 (Implement concat_bytes!)
 - rust-lang#89999 (Update std::env::temp_dir to use GetTempPath2 on Windows when available.)
 - rust-lang#90796 (Remove the reg_thumb register class for asm! on ARM)
 - rust-lang#91042 (Use Vec extend instead of repeated pushes on several places)
 - rust-lang#91634 (Do not attempt to suggest help for overly malformed struct/function call)
 - rust-lang#91685 (Install llvm tools to sysroot when assembling local toolchain)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 8c94b2c into rust-lang:master Dec 9, 2021
@rustbot rustbot added this to the 1.59.0 milestone Dec 9, 2021
@terrarier2111 terrarier2111 deleted the fix-recover-from-variant-ice branch December 24, 2021 09:00
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
5 participants