Skip to content

Account for proc macro spans in do_not_recommend diagnostics#156763

Open
qaijuang wants to merge 1 commit into
rust-lang:mainfrom
qaijuang:156676-follow-up
Open

Account for proc macro spans in do_not_recommend diagnostics#156763
qaijuang wants to merge 1 commit into
rust-lang:mainfrom
qaijuang:156676-follow-up

Conversation

@qaijuang
Copy link
Copy Markdown
Contributor

@qaijuang qaijuang commented May 19, 2026

Follow up to #156676.

Fixes #156759

r? @chenyukang

@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 May 19, 2026
fn finish(self) {}
}

// Regression test for https://github.com/rust-lang/rust/issues/156759.
Copy link
Copy Markdown
Member

@chenyukang chenyukang May 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prefer to add a standalone test case for a issue.

View changes since the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Regression in do_not_recommend span placement in macro-generated code

3 participants