Skip to content

Cleanup unused diagnostic emission methods - part 2#153509

Merged
rust-bors[bot] merged 8 commits intorust-lang:mainfrom
GuillaumeGomez:migrate-diag2
Mar 9, 2026
Merged

Cleanup unused diagnostic emission methods - part 2#153509
rust-bors[bot] merged 8 commits intorust-lang:mainfrom
GuillaumeGomez:migrate-diag2

Conversation

@GuillaumeGomez
Copy link
Member

@rustbot
Copy link
Collaborator

rustbot commented Mar 6, 2026

Some changes occurred in match checking

cc @Nadrieril

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

Some changes occurred to the CTFE / Miri interpreter

cc @rust-lang/miri, @RalfJung, @oli-obk, @lcnr

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 6, 2026
@rustbot rustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Mar 6, 2026
@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 7, 2026
@rustbot

This comment has been minimized.

@GuillaumeGomez
Copy link
Member Author

Introduced DiagDecorator to allow keep using closures if it makes more sense than a plain Diagnostic implementation.

Copy link
Contributor

@JonathanBrouwer JonathanBrouwer left a comment

Choose a reason for hiding this comment

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

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 8, 2026

📌 Commit 4f397b6 has been approved by JonathanBrouwer

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 8, 2026
rust-bors bot pushed a commit that referenced this pull request Mar 9, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #152535 (std: use `OnceLock` for Xous environment variables)
 - #152646 (Update `UnsafeUnpin` impls involving extern types.)
 - #153559 (Inline and simplify some code for saving incremental data to disk)
 - #151900 (num: Separate public API from internal implementations)
 - #153520 (.mailmap: fix broken line with multiple emails)
 - #153573 (rustdoc-json: fix incorrect documentation for VariantKind::Struct)

Failed merges:

 - #153509 (Cleanup unused diagnostic emission methods - part 2)
@rust-bors rust-bors bot added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Mar 9, 2026
@rust-bors

This comment has been minimized.

@rust-bors rust-bors bot removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 9, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 9, 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.

@rust-log-analyzer

This comment has been minimized.

@GuillaumeGomez
Copy link
Member Author

@bors r=JonathanBrouwer rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 9, 2026

📌 Commit d31a17f has been approved by JonathanBrouwer

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 9, 2026
rust-bors bot pushed a commit that referenced this pull request Mar 9, 2026
…uwer

Rollup of 11 pull requests

Successful merges:

 - #152847 (Test for armv7 `get_unchecked(...)` inlining)
 - #153290 (tests: codegen-llvm: iter-repeat-n-trivial-drop: Allow non-zero lower bound to __rust_alloc size)
 - #153413 (std: organise `sys::pal::os`)
 - #139692 (Rustfmt now support use closures)
 - #152800 (Adds recursion limit into FindParamInClause)
 - #153244 (tools: remote-test-server: Add UEFI run support)
 - #153305 (Adapt codegen test to accept operand bundles)
 - #153340 (tweak r-a default settings)
 - #153509 (Cleanup unused diagnostic emission methods - part 2)
 - #153527 (Fix LegacyKeyValueFormat report from docker build: ohos)
 - #153594 (Move some ui tests)
@rust-bors rust-bors bot merged commit d4630f1 into rust-lang:main Mar 9, 2026
11 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 9, 2026
rust-timer added a commit that referenced this pull request Mar 9, 2026
Rollup merge of #153509 - GuillaumeGomez:migrate-diag2, r=JonathanBrouwer

Cleanup unused diagnostic emission methods - part 2

Part of #153099.

r? @JonathanBrouwer
@GuillaumeGomez GuillaumeGomez deleted the migrate-diag2 branch March 9, 2026 16:47
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 9, 2026
…n_with_style, r=JonathanBrouwer

Add missing `Diag::with_span_suggestion_with_style` method

Uncovered while working on rust-lang#153509.

r? @JonathanBrouwer
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 9, 2026
…n_with_style, r=JonathanBrouwer

Add missing `Diag::with_span_suggestion_with_style` method

Uncovered while working on rust-lang#153509.

r? @JonathanBrouwer
rust-timer added a commit that referenced this pull request Mar 10, 2026
Rollup merge of #153609 - GuillaumeGomez:with_span_suggestion_with_style, r=JonathanBrouwer

Add missing `Diag::with_span_suggestion_with_style` method

Uncovered while working on #153509.

r? @JonathanBrouwer
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. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants