Skip to content

fix: Do not autoref in method probe in path mode#22392

Merged
ChayimFriedman2 merged 1 commit into
rust-lang:masterfrom
ChayimFriedman2:path-probe-autoref
May 20, 2026
Merged

fix: Do not autoref in method probe in path mode#22392
ChayimFriedman2 merged 1 commit into
rust-lang:masterfrom
ChayimFriedman2:path-probe-autoref

Conversation

@ChayimFriedman2
Copy link
Copy Markdown
Contributor

Fixes the first problem in #22387.

See also #t-types > Method resolution probe autorefs on path.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 17, 2026
140..153 'IsCopy.test()': bool
159..166 'NotCopy': NotCopy
159..173 'NotCopy.test()': bool
159..173 'NotCopy.test()': {unknown}
Copy link
Copy Markdown
Contributor Author

@ChayimFriedman2 ChayimFriedman2 May 17, 2026

Choose a reason for hiding this comment

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

The other test regressions are in path mode (where the trait doesn't hold) so expected. The ones in this file is in method call mode, so less expected. The reason for it is that if we fail to resolve with method call mode we retry with path mode.

rustc fails to resolve these methods as well.

View changes since the review

@rustbot

This comment has been minimized.

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 20, 2026

This PR was rebased onto a different master 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.

@ChayimFriedman2 ChayimFriedman2 enabled auto-merge May 20, 2026 14:17
@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue May 20, 2026
Merged via the queue into rust-lang:master with commit 2ba9e2e May 20, 2026
18 checks passed
@ChayimFriedman2 ChayimFriedman2 deleted the path-probe-autoref branch May 20, 2026 14:40
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 20, 2026
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