Skip to content

Conversation

mitaa and others added 11 commits December 15, 2015 01:55
@ubsan brought up this relatively poor error message. This adds a
help message hinting when the problem actually is, and how to fix
it.
None of my devices/browsers renders this correctly.

r? @steveklabnik
…=Manishearth

@ubsan brought up this relatively poor error message. This adds a
help message hinting when the problem actually is, and how to fix
it.
Also, make its panic description identical in wording to assert_eq.

Fixes rust-lang#30600
@steveklabnik
Copy link
Contributor Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Dec 29, 2015

📌 Commit 5a02d22 has been approved by steveklabnik

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

bors added a commit that referenced this pull request Dec 29, 2015
@bors
Copy link
Collaborator

bors commented Dec 29, 2015

⌛ Testing commit 5a02d22 with merge c1035b3...

1 similar comment
@bors bors merged commit 5a02d22 into rust-lang:master Dec 30, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollup A PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants