Skip to content

Conversation

@Human9000-bit
Copy link
Contributor

  • test: modified bad-question-mark-on-trait-objects to match expected behavior
  • removed confusing message from diagnostics

fixes #150527

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 1, 2026
@Human9000-bit Human9000-bit marked this pull request as ready for review January 1, 2026 20:13
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 1, 2026
@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jan 1, 2026
@rustbot
Copy link
Collaborator

rustbot commented Jan 1, 2026

r? @jieyouxu

rustbot has assigned @jieyouxu.
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

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Hm okay. I was wondering why the reverse suggestion doesn't work, then I remembered the try desugaring uses the From impl not the Into impl -- and we can't make that change because it has ugly inference breakages, cf. #60796

View changes since this review

@jieyouxu
Copy link
Member

jieyouxu commented Jan 3, 2026

Thanks
@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 3, 2026

📌 Commit 97e5cab has been approved by jieyouxu

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 Jan 3, 2026
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jan 3, 2026
Removed confusing diagnostics note for trait required for `?` operator use

- **test: modified `bad-question-mark-on-trait-objects` to match expected behavior**
- **removed confusing message from diagnostics**

fixes [rust-lang#150527](rust-lang#150527)
bors added a commit that referenced this pull request Jan 3, 2026
Rollup of 5 pull requests

Successful merges:

 - #150558 (Detect cases where `?` is applied on a type that could be coming from a different crate version than expected)
 - #150570 (Removed confusing diagnostics note for trait required for `?` operator use)
 - #150589 (Remove unneeded `forbid_generic` field from `Res::SelfTyAlias`)
 - #150635 (triagebot: Add a mention for `dec2flt`, `flt2dec`, and `fmt/num.rs`)
 - #150637 (Fix broken edit links in Unstable and rustc books)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 7af208f into rust-lang:main Jan 3, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 3, 2026
rust-timer added a commit that referenced this pull request Jan 3, 2026
Rollup merge of #150570 - Human9000-bit:main, r=jieyouxu

Removed confusing diagnostics note for trait required for `?` operator use

- **test: modified `bad-question-mark-on-trait-objects` to match expected behavior**
- **removed confusing message from diagnostics**

fixes [#150527](#150527)
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.

Confusing diagnostic on ? operator: needs to implement Into<Error>

4 participants