Skip to content

Adjust orphan rules to consider all input types, not just self type. - #18223

Closed
nikomatsakis wants to merge 1 commit into
rust-lang:masterfrom
nikomatsakis:coherence-orphan-18222
Closed

Adjust orphan rules to consider all input types, not just self type.#18223
nikomatsakis wants to merge 1 commit into
rust-lang:masterfrom
nikomatsakis:coherence-orphan-18222

Conversation

@nikomatsakis

Copy link
Copy Markdown
Contributor

Adjust orphan rules to consider all input types, not just self type.

Fixes#18222.

r? @pcwalton

@nikomatsakis

Copy link
Copy Markdown
ContributorAuthor

cc @aturon this fixes that coherence failure we saw with the Index trait

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Now that is some readable code :-)

bors added a commit that referenced this pull request Oct 22, 2014
…walton
Adjust orphan rules to consider all input types, not just self type.
Fixes#18222.
r? @pcwalton
@borsbors closed this Oct 22, 2014
@nikomatsakis
nikomatsakis deleted the coherence-orphan-18222 branch March 30, 2016 16:12
lnicola pushed a commit to lnicola/rust that referenced this pull request Oct 8, 2024
…lbasi
Use external stack in borrowck DFS
Because damnit, it can crash r-a. Why do people make this stupid DFSes anyway (I get it, it's easier until it blows).
Fixesrust-lang#18223 (who thought DFS will be the problem).
Sign up for freeto 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.

Orphan rules in coherence do not account for multidispatch

4 participants

@nikomatsakis@pcwalton@aturon@bors