Skip to content

Catch attempts to project nonexistent associated items from a trait.#22759

Closed
pnkfelix wants to merge 1 commit into
rust-lang:masterfrom
pnkfelix:issue-22731-proj-nonexist-assoc-item
Closed

Catch attempts to project nonexistent associated items from a trait.#22759
pnkfelix wants to merge 1 commit into
rust-lang:masterfrom
pnkfelix:issue-22731-proj-nonexist-assoc-item

Conversation

@pnkfelix

Copy link
Copy Markdown
Contributor

Catch attempts to project nonexistent associated items from a trait.

(Also, fix an instance of this mistake this fix uncovered in iter.rs)

Fix #22731.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @eddyb

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

@pnkfelix
pnkfelix force-pushed the issue-22731-proj-nonexist-assoc-item branch from 1adb599 to 3a60f2c Compare February 24, 2015 10:32
@eddyb

eddyb commented Feb 24, 2015

Copy link
Copy Markdown
Contributor

This is obsoleted by #22172, which is in the queue.

@eddyb eddyb closed this Feb 24, 2015
@pnkfelix

Copy link
Copy Markdown
Contributor Author

(of course, my PR might actually land. :( )

((sorry for the snark, I'm really more just upset by how badly our overall infrastructure is behaving right now...))

@eddyb

eddyb commented Feb 24, 2015

Copy link
Copy Markdown
Contributor

@pnkfelix I deserve around 35% of that snark. Feel free to reopen and r=me if it doesn't merge today.

lnicola pushed a commit to lnicola/rust that referenced this pull request Jul 20, 2026
…pping-unwraps

Fix crashes in assists due to .unwrap() calls in SyntaxFactory
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.

Projecting <Type as Trait>::N should check that N is part of Trait

3 participants