Skip to content

Comments

rustc_expand: improve diagnostics for non-repeatable metavars#152679

Open
Unique-Usman wants to merge 1 commit intorust-lang:mainfrom
Unique-Usman:ua/decmacrounrepeatable
Open

rustc_expand: improve diagnostics for non-repeatable metavars#152679
Unique-Usman wants to merge 1 commit intorust-lang:mainfrom
Unique-Usman:ua/decmacrounrepeatable

Conversation

@Unique-Usman
Copy link
Contributor

@Unique-Usman Unique-Usman commented Feb 15, 2026

Fix #47452.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 15, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 15, 2026

r? @BoxyUwU

rustbot has assigned @BoxyUwU.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 68 candidates
  • Random selection from 14 candidates

@Unique-Usman
Copy link
Contributor Author

r? @estebank

@rustbot rustbot assigned estebank and unassigned BoxyUwU Feb 15, 2026
@Unique-Usman
Copy link
Contributor Author

This is rebased on top of ->791ee34

@rust-log-analyzer

This comment has been minimized.

@Unique-Usman Unique-Usman force-pushed the ua/decmacrounrepeatable branch from 0fbe120 to 81f68bb Compare February 15, 2026 21:33
@rust-bors

This comment has been minimized.

@Unique-Usman Unique-Usman force-pushed the ua/decmacrounrepeatable branch from 81f68bb to 60060dd Compare February 20, 2026 14:59
@rustbot
Copy link
Collaborator

rustbot commented Feb 20, 2026

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@Unique-Usman Unique-Usman force-pushed the ua/decmacrounrepeatable branch from 9704509 to d4e8858 Compare February 23, 2026 20:17
@rust-log-analyzer

This comment has been minimized.

Co-authored-by: Esteban Küber <esteban@kuber.com.ar>
Signed-off-by: Usman Akinyemi <usmanakinyemi202@gmail.com>
@Unique-Usman
Copy link
Contributor Author

@estebank - this is ready to land. Thank you for the review.

@estebank
Copy link
Contributor

@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 24, 2026

📌 Commit 93b9973 has been approved by estebank

It is now in the queue for this repository.

@rust-bors rust-bors bot 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 Feb 24, 2026
@Unique-Usman
Copy link
Contributor Author

this #47452 should be closed now ?

@estebank
Copy link
Contributor

Updated the description so that when the PR merges the ticket gets autoclosed.

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 24, 2026
…le, r=estebank

rustc_expand: improve diagnostics for non-repeatable metavars

Fix rust-lang#47452.
rust-bors bot pushed a commit that referenced this pull request Feb 25, 2026
Rollup of 8 pull requests

Successful merges:

 - #149169 (ptr::replace: make calls on ZST null ptr not UB)
 - #150562 (Fix doc link used in suggestion for pinning self)
 - #152679 (rustc_expand: improve diagnostics for non-repeatable metavars)
 - #153017 (Implement debuginfo for unsafe binder types)
 - #152868 (delete some very old trivial `Box` tests)
 - #152922 (rustc_public: Make fields that shouldn't be exposed visible only in `rustc_public`)
 - #153029 (Rename `rustc::pass_by_value` lint as `rustc::disallowed_pass_by_ref`.)
 - #153051 (Migration of `LintDiagnostic` - part 3)
jhpratt added a commit to jhpratt/rust that referenced this pull request Feb 25, 2026
…le, r=estebank

rustc_expand: improve diagnostics for non-repeatable metavars

Fix rust-lang#47452.
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.

macro_rules: "no syntax variables matched as repeating at this depth" fires before "unknown macro variable"

5 participants