-
Notifications
You must be signed in to change notification settings - Fork 12.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
only find segs chain for missing methods when no available candidates #125080
Conversation
r? @nnethercote rustbot has assigned @nnethercote. Use |
@rustbot ready |
r=me once the typo is fixed. For future reference, a nice way to do this kind of PR is to add the new test in a first commit, and then make the change to the error message in the second commit. That way the reviewer can easily see exactly how the change improves the error output. @bors delegate=bvanjoi |
✌️ @bvanjoi, you can now approve this pull request! If @nnethercote told you to " |
…mpiler-errors Rollup of 7 pull requests Successful merges: - rust-lang#119838 (style-guide: When breaking binops handle multi-line first operand better) - rust-lang#124844 (Use a proper probe for shadowing impl) - rust-lang#125047 (Migrate `run-make/issue-14500` to new `rmake.rs` format) - rust-lang#125080 (only find segs chain for missing methods when no available candidates) - rust-lang#125088 (Uplift `AliasTy` and `AliasTerm`) - rust-lang#125100 (Don't do post-method-probe error reporting steps if we're in a suggestion) - rust-lang#125118 (Use new utility functions/methods in run-make tests) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#125080 - bvanjoi:fix-124946, r=nnethercote only find segs chain for missing methods when no available candidates Fixes rust-lang#124946 This PR includes two changes: - Extracting the lookup for the missing method in chains into a single function. - Calling this function only when there are no candidates available.
Fixes #124946
This PR includes two changes: