Skip to content

Add E0036 error explanation#26400

Closed
GuillaumeGomez wants to merge 2 commits into
rust-lang:masterfrom
GuillaumeGomez:patch-6
Closed

Add E0036 error explanation#26400
GuillaumeGomez wants to merge 2 commits into
rust-lang:masterfrom
GuillaumeGomez:patch-6

Conversation

@GuillaumeGomez
Copy link
Copy Markdown
Member

Part of #24407.
cc @michaelsproul

@rust-highfive
Copy link
Copy Markdown
Contributor

r? @Aatch

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

Comment thread src/librustc_typeck/diagnostics.rs Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

“Bad example” suggests that this example itself is of unsatisfactory quality, while this example simply demonstrates how such an error might be caused. I suggest dropping “Bad” here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I agree, I remove it!

@GuillaumeGomez
Copy link
Copy Markdown
Member Author

@nagisa: Updated, thanks for your help! Do you see anything else?

@pnkfelix
Copy link
Copy Markdown
Contributor

@bors r+

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jun 19, 2015

📌 Commit 222e127 has been approved by pnkfelix

@pnkfelix
Copy link
Copy Markdown
Contributor

@bors r+

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jun 19, 2015

📌 Commit 222e127 has been approved by pnkfelix

@pnkfelix
Copy link
Copy Markdown
Contributor

@bors r+

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jun 19, 2015

📌 Commit 38950ae has been approved by pnkfelix

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.

6 participants