Skip to content

Small clean-ups#21829

Merged
lnicola merged 1 commit intorust-lang:masterfrom
ada4a:push-zwprmlkyptmx
Mar 16, 2026
Merged

Small clean-ups#21829
lnicola merged 1 commit intorust-lang:masterfrom
ada4a:push-zwprmlkyptmx

Conversation

@ada4a
Copy link
Contributor

@ada4a ada4a commented Mar 16, 2026

Some things I've noticed while working on https://rust-lang.zulipchat.com/#narrow/channel/185405-t-compiler.2Frust-analyzer/topic/Type.20inference.20on.20non-speculative.20file.20fails.20--.20due.20to.20.60.2E.2E.60.3F/with/579457494, which I wasn't able to finish, hence this separate small PR.

  • extract unrelated if out of matches!
  • in include_references:
    • add docs
    • use a while-let chain
    • add a more complex test

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 16, 2026
postfix_completion_for_block("const");
}

#[track_caller]
Copy link
Member

Choose a reason for hiding this comment

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

Isn't it useful to know which check_edit failed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, you're right, #[track_caller] removes that information. Not sure what I was thinking here. Removed the change

- extract unrelated `if` out of `matches!`
- in `include_references`:
  - add docs
  - use a while-let chain
  - add a more complex test
@ada4a ada4a force-pushed the push-zwprmlkyptmx branch from 3ca6790 to ec677f8 Compare March 16, 2026 20:22
@lnicola lnicola enabled auto-merge March 16, 2026 20:24
@lnicola lnicola added this pull request to the merge queue Mar 16, 2026
Merged via the queue into rust-lang:master with commit 80b71d4 Mar 16, 2026
17 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 16, 2026
@ada4a ada4a deleted the push-zwprmlkyptmx branch March 16, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants