Skip to content
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

Fix regression: account for impl methods in arg count mismatch error #47747

Merged
merged 1 commit into from
Jan 26, 2018

Conversation

estebank
Copy link
Contributor

Fix #47706.

@estebank estebank added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 25, 2018
@rust-highfive
Copy link
Contributor

r? @petrochenkov

(rust_highfive has picked a reviewer for you, use r? to override)

@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 25, 2018

📌 Commit 6006b67 has been approved by petrochenkov

@petrochenkov petrochenkov 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 25, 2018
@estebank
Copy link
Contributor Author

@bors rollup

@nikomatsakis
Copy link
Contributor

I don't think this should be a rollup -- it fixes a regression, after all.

@nikomatsakis
Copy link
Contributor

nikomatsakis commented Jan 26, 2018

Well, I guess it depends what rollup means. I tend to think of it as "low risk and it lands when it lands", but maybe that's not quite right.

@estebank
Copy link
Contributor Author

My rationale was that it had a better chance of landing sooner without giving it a higher priority, given that it is a small change and didn't want to take a while slot for it. Feel free to change it removing the rollup and increasing the priority.

@bors bors merged commit 6006b67 into rust-lang:master Jan 26, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Jan 31, 2018
Fix regression: account for trait methods in arg count mismatch error

Fixed rust-lang#47706 (rust-lang#47706 (comment))

Original PR rust-lang#47747 missed methods on trait definitions.

This edit was done in GitHub. I think I got the signature of the variant right, going by the ICE debug output and the other cases above.
@estebank estebank deleted the fix-regression branch November 9, 2023 05:25
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants