Skip to content

Conversation

jmqd
Copy link
Contributor

@jmqd jmqd commented Jul 23, 2022

Extremely minor QOL change to improve the ICE compiler output in case
this default match case is encountered (an unexpected rib kind).

I have limited experience in this area of the compiler; please let me know
if a span more precise than param.ident.span is more applicable.

Extremely minor QOL change to improve the ICE output in case this
default match case is encountered (an unexpected rib kind).
@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jul 23, 2022
@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @oli-obk (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 23, 2022
@oli-obk
Copy link
Contributor

oli-obk commented Jul 23, 2022

The span seems close enough to me, let's use it for now

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 23, 2022

📌 Commit e0c9be5 has been approved by oli-obk

It is now in the queue for this repository.

@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 Jul 23, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 23, 2022
…askrgr

Rollup of 4 pull requests

Successful merges:

 - rust-lang#99580 (Don't suggest unnameable generic arguments)
 - rust-lang#99617 (Update mdbook)
 - rust-lang#99631 (Use span_bug in case of unexpected rib kind)
 - rust-lang#99632 (Fix typo/grammar in locator.rs doc comment)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit f510d98 into rust-lang:master Jul 23, 2022
@rustbot rustbot added this to the 1.64.0 milestone Jul 23, 2022
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants